search

AmpersandJS / amp

a collection of individual JS utility modules
http://amp.ampersandjs.com
MIT License
130 stars 12 forks source link

adding array-to-readable #91

Closed HenrikJoreteg closed 9 years ago

wraithgar commented 9 years ago

Couldn't this be amp-array-join if you made the separator also configurable (defaulting to , still)?

HenrikJoreteg commented 9 years ago

@wraithgar only thing is, the fact that it joins the stuff isn't what's useful, IMO. It's the fact that it's producing a human friendly string. What else would you use it for?

bear commented 9 years ago

Could you use amp-array-join with the default of "," and a flag for human readable?

In the python world .join() is used for this case (as an example)

HenrikJoreteg commented 9 years ago

@bear while i could, there's really not much reason to do so. Given that Array.prototype.join exists and does exactly that. If that's all you want, you wouldn't use this module.

wraithgar commented 9 years ago

+1 my question was answered