Drop aggregate-error and require node >=16

sindresorhus / p-map

Map over promises concurrently

MIT License

1.27k stars 58 forks source link

Drop aggregate-error and require node >=16 #65

Closed furudean closed 1 year ago

furudean commented 1 year ago

See rationale in #64

fixes #64 fixes #59

furudean commented 1 year ago

The linter wants an error message string. I guess this only gets picked up on the native AggregateError. Should we just ignore this, or add a message? @sindresorhus

sindresorhus commented 1 year ago

Ignore it

silverwind commented 1 year ago

Could wait until 2023-04-18 with this when Node 20 releases, so support for 20 and requirement of 16 can be done in one go.

sindresorhus commented 1 year ago

Yes, I don't plan to merge this until then.

silverwind commented 1 year ago

Time for merge?