Current project status about pump HOT 6 CLOSED

Yup, in Node.js 10 pipeline was added (by me :)) to have pump in core. I still use pump personally whenever I want to support Node.js <10

from pump.

Yep!

from pump.

Good to hear. Am I correct though that beginning with Node 10 and stream.pipeline pump is not longer required ? If so i think it would be appreciated if the README commented on to why or why not ?

from pump.

@matthiasg you are correct - the following quote from the Node docs confirms it:

pump is only necessary for Nodejs 8.x or earlier, as for Node 10.x or later version, pipeline is introduced to replace for pump. This is a module method to pipe between streams forwarding errors and properly cleaning up and provide a callback when the pipeline is complete.

from pump.

@mafintosh @lukesims sorry had actually read it in the node docs, but had forgotten about it. It would be sensible to mention in this projects README as well though, to close the loop for people coming from other projects (like I did) trying to figure out whether its required. good work btw

from pump.

I agree with @matthiasg as it would save future users from making the same side-trip to the Node docs that we all did to reach this conclusion. This is the PR if anyone else is interested - good work @mafintosh :)

from pump.