Drop dependency on readable-stream

[JustinBeckwith]

It feels like the implementation of streams in Node.js 6+ has settled down a great deal. Would you consider a PR to drop the dependency on the user space implementation of readable-stream? Or are there good reasons to keep on using it?

[phated]

https://r.va.gg/2014/06/why-i-dont-use-nodes-core-stream-module.html

[JustinBeckwith]

Thanks! I totally understand the advantage of the readable-stream package.. a few years ago :) A lot of what Rod wrote about was true in the pre-4.x world. As of today though, I'd argue that streams have settled. The API seems fairly locked in, and functionality is pretty much the same across Node 6 => 11 AFAICT. I am absolutely not the expert here though :)

[mafintosh]

Being part of the streams working group readable-stream is still very much needed as there and bigger changes still happening underneat the hood of streams

[JustinBeckwith]

Thanks! Can you give a TL;DR of what those changes look like? We're trying to evaluate our usage of streams across all of the Google npm modules, and I find myself often conflicted 😁

[phated]

I'll let mafintosh speak to changes but I'm going to close this so it doesn't show up on my github dashboard.