[v12.x backport] stream: avoid unecessary nextTick

[ronag]

Refs: #29194

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

[ronag]

This is my first backport. I don't understand what Backport-PR-URL: is supposed to contain (step 9.v?). URL to this link or the original PR?

[ronag]

Also I don't believe I can do 9.vi? Can I?

[ronag]

ping @BridgeAR

[targos]

Backport-PR-URL should contain this PR's url, so:

Backport-PR-URL: https://github.com/nodejs/node/pull/29691

[targos]

It looks like the commit metadata from the original was lost. This should also be in the commit message:

PR-URL: https://github.com/nodejs/node/pull/29194
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/25679/

[ronag]

Unrelated travis failure?

[BridgeAR]

@ronag was there a specific reason that you closed the PR? The releasers normally land the backport PRs some time before preparing the next release. Seems like it did not end up in the last release and would need another rebase.

The travis failure should be ignored as long as the Node.js CI passes.

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/26470/

[ronag]

This PR #30851 pretty much makes this redudant. I'll make a new backport port based on #30851 once that lands.