Skip to content

stream: make sure 'readable' is emitted before ending the stream

Rodrigo Muino Tomonari requested to merge github/fork/mcollina/readable-next-tick into master

This change is born out of https://github.com/nodejs/node/issues/25810, as I explored how to fix that difference.

This change could be considered semver-major or a bug fix. We need also to check some of the benchmarks if they are impacted before landing.

Fixes https://github.com/nodejs/node/issues/25810

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines

Merge request reports