sindresorhus / p-map

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

Add `pMapSkip` #39

Closed Richienb closed 3 years ago

Richienb commented 3 years ago

Fixes #38

sindresorhus commented 3 years ago

I'm not sold on calling it just skip when it's a named export. Either a more verbose name or put it on pMap. I think I would prefer the former.

Richienb commented 3 years ago

I'm not sold on calling it just skip when it's a named export. Either a more verbose name or put it on pMap. I think I would prefer the former.

Maybe pMapSkip?

sindresorhus commented 3 years ago

Maybe pMapSkip?

Yeah, I cannot think of anything better.

sindresorhus commented 3 years ago

Can you fix the merge conflict?

Richienb commented 3 years ago

@sindresorhus Could you release this for now?

sindresorhus commented 3 years ago

https://github.com/sindresorhus/p-map/releases/tag/v5.1.0