stream: add errored and closed props #40696
Conversation
ronag
force-pushed
the
stream-props
branch
3 times, most recently
from
0bf098c to
998dd7c
Compare
|
@nodejs/streams |
ronag
force-pushed
the
stream-props
branch
2 times, most recently
from
6328b07 to
5c04583
Compare
ronag
added
request-ci
github-actions
bot
removed
the
request-ci
ronag
added
the
author ready
|
|
||
| * {boolean} | ||
|
|
||
| Is `true` after `'close'` has been emitted. |
There was a problem hiding this comment.
Nit: Is it also true while it is being emitted? Personally, I'd interpret "after 'close' has been emitted" as in the following snippet, but that's probably not what is meant.
let value = false;
const e = new (require('events').EventEmitter)();
e.addListener('close', () => console.log(value)); // prints false
e.emit('close');
// *after* close has been emitted
value = true;There was a problem hiding this comment.
true enough, but that's how the other props that work similarly are also documented.
|
|
||
| <!-- YAML | ||
| added: v8.0.0 | ||
| --> | ||
|
|
||
| * {boolean} | ||
|
|
||
| Is `true` after `'close'` has been emitted. |
|
This should probably have a CITGM run. |
ronag
added
semver-major
ronag
added
request-ci
github-actions
bot
removed
the
request-ci
ronag
added
the
commit-queue
nodejs-github-bot
added
commit-queue-failed
Commit Queue failed- Loading data for nodejs/node/pull/40696 ✔ Done loading data for nodejs/node/pull/40696 ----------------------------------- PR info ------------------------------------ Title stream: add errored and closed props (#40696) Author Robert Nagy (@ronag) Branch ronag:stream-props -> nodejs:master Labels stream, semver-major, author ready Commits 4 - stream: add errored and closed props - fixup - fixup - fixup Committers 1 - Robert Nagy PR-URL: https://github.com/nodejs/node/pull/40696 Reviewed-By: Matteo Collina Reviewed-By: Benjamin Gruenbaum Reviewed-By: James M Snell ------------------------------ Generated metadata ------------------------------ PR-URL: https://github.com/nodejs/node/pull/40696 Reviewed-By: Matteo Collina Reviewed-By: Benjamin Gruenbaum Reviewed-By: James M Snell -------------------------------------------------------------------------------- ℹ This PR was created on Tue, 02 Nov 2021 10:27:13 GMT ✔ Approvals: 3 ✔ - Matteo Collina (@mcollina) (TSC): https://github.com/nodejs/node/pull/40696#pullrequestreview-799321929 ✔ - Benjamin Gruenbaum (@benjamingr): https://github.com/nodejs/node/pull/40696#pullrequestreview-795708212 ✔ - James M Snell (@jasnell) (TSC): https://github.com/nodejs/node/pull/40696#pullrequestreview-802483347 ✔ Last GitHub Actions successful ℹ Last Full PR CI on 2021-11-10T09:49:58Z: https://ci.nodejs.org/job/node-test-pull-request/40812/ ℹ Last CITGM CI on 2021-11-06T18:42:52Z: https://ci.nodejs.org/view/Node.js-citgm/job/citgm-smoker/2790/ - Querying data for job/node-test-pull-request/40812/ ✔ Last Jenkins CI successful -------------------------------------------------------------------------------- ✔ No git cherry-pick in progress ✔ No git am in progress ✔ No git rebase in progress -------------------------------------------------------------------------------- - Bringing origin/master up to date... From https://github.com/nodejs/node * branch master -> FETCH_HEAD 2d368da19f..8ee9e1a1aa master -> origin/master ✔ origin/master is now up-to-date master is out of sync with origin/master. Mismatched commits: - a2ae06435b src,crypto: refactor `crypto_tls.*` - 8ee9e1a1aa src,crypto: refactor `crypto_tls.*` -------------------------------------------------------------------------------- HEAD is now at 8ee9e1a1aa src,crypto: refactor `crypto_tls.*` ✔ Reset to origin/master - Downloading patch for 40696 From https://github.com/nodejs/node * branch refs/pull/40696/merge -> FETCH_HEAD ✔ Fetched commits as 8ee9e1a1aaf5..f4416e00b290 -------------------------------------------------------------------------------- Auto-merging doc/api/stream.md [master 6efdeb1fd5] stream: add errored and closed props Author: Robert Nagy Date: Tue Nov 2 12:01:48 2021 +0200 7 files changed, 110 insertions(+), 7 deletions(-) [master e35f3a9bcc] fixup Author: Robert Nagy Date: Thu Nov 4 23:00:13 2021 +0200 1 file changed, 6 deletions(-) [master b703ad992e] fixup Author: Robert Nagy Date: Fri Nov 5 22:50:05 2021 +0200 1 file changed, 1 insertion(+), 1 deletion(-) [master 653829552e] fixup Author: Robert Nagy Date: Fri Nov 5 22:50:45 2021 +0200 1 file changed, 1 insertion(+), 1 deletion(-) ✔ Patches applied There are 4 commits in the PR. Attempting autorebase. Rebasing (2/8)https://github.com/nodejs/node/actions/runs/1436426233 |
aduh95
added
commit-queue
nodejs-github-bot
removed
the
commit-queue
Commit Queue failed- Loading data for nodejs/node/pull/40696 ✔ Done loading data for nodejs/node/pull/40696 ----------------------------------- PR info ------------------------------------ Title stream: add errored and closed props (#40696) Author Robert Nagy (@ronag) Branch ronag:stream-props -> nodejs:master Labels stream, semver-major, author ready, commit-queue-squash Commits 4 - stream: add errored and closed props - fixup - fixup - fixup Committers 1 - Robert Nagy PR-URL: https://github.com/nodejs/node/pull/40696 Reviewed-By: Matteo Collina Reviewed-By: Benjamin Gruenbaum Reviewed-By: James M Snell ------------------------------ Generated metadata ------------------------------ PR-URL: https://github.com/nodejs/node/pull/40696 Reviewed-By: Matteo Collina Reviewed-By: Benjamin Gruenbaum Reviewed-By: James M Snell -------------------------------------------------------------------------------- ℹ This PR was created on Tue, 02 Nov 2021 10:27:13 GMT ✔ Approvals: 3 ✔ - Matteo Collina (@mcollina) (TSC): https://github.com/nodejs/node/pull/40696#pullrequestreview-799321929 ✔ - Benjamin Gruenbaum (@benjamingr): https://github.com/nodejs/node/pull/40696#pullrequestreview-795708212 ✔ - James M Snell (@jasnell) (TSC): https://github.com/nodejs/node/pull/40696#pullrequestreview-802483347 ✔ Last GitHub Actions successful ℹ Last Full PR CI on 2021-11-12T21:48:13Z: https://ci.nodejs.org/job/node-test-pull-request/40812/ ℹ Last CITGM CI on 2021-11-12T21:48:13Z: https://ci.nodejs.org/view/Node.js-citgm/job/citgm-smoker/2790/ - Querying data for job/node-test-pull-request/40812/ ✔ Last Jenkins CI successful -------------------------------------------------------------------------------- ✔ No git cherry-pick in progress ✔ No git am in progress ✔ No git rebase in progress -------------------------------------------------------------------------------- - Bringing origin/master up to date... From https://github.com/nodejs/node * branch master -> FETCH_HEAD ✔ origin/master is now up-to-date - Downloading patch for 40696 From https://github.com/nodejs/node * branch refs/pull/40696/merge -> FETCH_HEAD ✔ Fetched commits as d10085bcb1f4..f4416e00b290 -------------------------------------------------------------------------------- Auto-merging doc/api/stream.md [master 110fb7c8f1] stream: add errored and closed props Author: Robert Nagy Date: Tue Nov 2 12:01:48 2021 +0200 7 files changed, 110 insertions(+), 7 deletions(-) [master 53b5a15660] fixup Author: Robert Nagy Date: Thu Nov 4 23:00:13 2021 +0200 1 file changed, 6 deletions(-) [master 1eadfb9bc3] fixup Author: Robert Nagy Date: Fri Nov 5 22:50:05 2021 +0200 1 file changed, 1 insertion(+), 1 deletion(-) [master 5526975255] fixup Author: Robert Nagy Date: Fri Nov 5 22:50:45 2021 +0200 1 file changed, 1 insertion(+), 1 deletion(-) ✔ Patches applied There are 4 commits in the PR. Attempting autorebase. Rebasing (2/8)https://github.com/nodejs/node/actions/runs/1454898010 |
nodejs-github-bot
added
the
commit-queue-failed
|
Landed in f217025 |
No description provided.