Probably worth a semver-major bump to 4.0.0, it'll probably not work in Node 6 (maybe?) if anyone's still relying on that, and the removal of the custom destroy() and replacing it with the Writable#destroy() could have some breaking effects due to the additional complexity of the latter. Mostly it should be a straightforward upgrade for anyone still using through2.

I think I might remove readable-stream@2 option and just make the dependency "3" (it is currently "2 || 3"), it's about time to move on and encourage others to do so. Any objections?

@rvagg +1

Following your lead 👍

• switched the docs to point to readable-stream
• cleaned up test/bench.js
• explicitly require Buffer (in prep for the webpack@5 storm)

published as 4.0.0, then realised .npmignore was out of date and there was cruft in the pack so I fixed that up and pushed a 4.0.1