nodejs / readable-stream

Node-core streams for userland
https://nodejs.org/api/stream.html
Other
1.03k stars 225 forks source link

fix: remove unused `require('stream')` and extend webpack tests #519

Closed jeswr closed 1 year ago

jeswr commented 1 year ago

Closes #516

Potentially supercedes #517 by also including a fix; which is removing require('stream') in https://github.com/jeswr/readable-stream/blob/3caace80d34bacaeb3d2aa760f3bd0fbdd9ae3d3/lib/stream/promises.js.

However this comment makes me suspicious of whether this one-liner is the desired fix - hence why I have opened this as a separate PR.