Upgrade semver of readable-stream from 3 to ^4.0.0

[Annniee1]

Upgrade semver of readable-stream from 3 to 4.2.0

[Annniee1]

@brycebaril @maxogden @rvagg @styfle Please Have a look at this PR

[styfle]

Suggested change

	"readable-stream": "4.2.0"
	"readable-stream": "4"

[Annniee1]

Changes Done. Please Review It @styfle

[styfle]

Seems fine although I don't have any authority here and didn't run the tests

[Annniee1]

@rvagg Please Review this PR

[rvagg]

🤦 CI is very out of date ...

for a start, let's go with ^4.2.0, but secondly we need to fix CI, which means ditching 8.x->12.x; perhaps copy this matrix: https://github.com/rvagg/bl/blob/master/.github/workflows/test-and-release.yml#L8

also the browser tests are broken, probably a bit of my fault too since it uses polendina .. which is mine

and then, after all that, we'll need to semver-major this because of the drop of support for older version of Node.js

[Annniee1]

Thanks for the detailed response @rvagg . Do we have some ETA for getting this fixed?
We are using string_decoder in our project,. In our project, We are getting compliance issues for string_decoder dependency which is getting pulled this way, through2 -> readable-stream -> string_decoder.
Do we have some way to get the required version of through2 which pulls readable-stream@4.2.0.
Looking forward to get some more understanding regarding this.
@rvagg @Delapouite @maxogden @styfle

[legobeat]

@rvagg Is this repo still under maintenance / is this issue something that can be addressed / do you need help?

[rvagg]

if someone wants to help push this forward, I'd like to switch CI to use the same as in bl, https://github.com/rvagg/bl/tree/master/.github/workflows; having a failing CI means I can't merge this