search

sindresorhus / p-map

Map over promises concurrently
MIT License
1.27k stars 58 forks source link

Don't convert input to an array #1

Closed sindresorhus closed 7 years ago

sindresorhus commented 7 years ago

https://github.com/sindresorhus/p-map/blob/051cf2692fb66dbc05a2ef41101c7fcf6e108207/index.js#L3

The iterable should be used directly instead of converting to an array. It would have the benefit of better supporting generated sequences.

A pull request would be wonderful 🙌🌟

parro-it commented 7 years ago

Awesome work with all the promise repos :cool: :+1: I would like to write a PR, I was thinking to suggest you the same change while reading your code, then I saw this issue :smiley: