search

teambition / merge2

Merge multiple streams into one stream in sequence or parallel (~119M/month downloads).
MIT License
170 stars 14 forks source link

Require "node:stream" instead of "stream" #40

Open boenrobot opened 3 months ago

boenrobot commented 3 months ago

And adjust minimum nodejs version accordingly.

Also removed package-lock.json and added it to .gitignore.

Using "node:stream" instead of "stream" enable use of this library in runtimes other than nodejs (e.g. workerd), at the cost of leaving behind older Nodejs versions.

The package-lock.json is unnecessary in a library, especially when not in a monorepo under CI. It generated a lockfile with older versions due to me testing at 14.18.0 and its bundled npm, while I imagine the latest used was newer.