Skip to content

doc: document major finished changes in v14 #33065

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
ronag wants to merge 5 commits into from
Closed

doc: document major finished changes in v14 #33065

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
9392877
doc: document major finished changes in v14
ronag Apr 25, 2020
ba8cfb6
fixup: premature close does not apply to pipeline due to compat worka…
ronag Apr 25, 2020
fea0924
fixup: formatting
ronag Apr 27, 2020
644278a
fixup: indent
ronag Apr 27, 2020
b4eea45
fixup: linting
ronag Apr 27, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fixup: formatting
  • Loading branch information
@ronag
ronag committed Apr 27, 2020
commit fea09242c017676e473bf4579075d4850e820a4c
16 changes: 8 additions & 8 deletions doc/api/stream.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1507,17 +1507,17 @@ changes:
- version: v14.0.0
pr-url: https://github.com/nodejs/node/pull/32158
description: `finished(stream, cb)` will wait for the `'close'` event
before invoking the callback. The implementation tries to detect legacy
streams and only apply this behavior to streams which are expected
to emit `'close'`.
before invoking the callback. The implementation tries to
detect legacy streams and only apply this behavior to streams
which are expected to emit `'close'`.
- version: v14.0.0
pr-url: https://github.com/nodejs/node/pull/31545
description: Emitting `'close'` before `'end'` on a `Readable` stream
will cause an `ERR_STREAM_PREMATURE_CLOSE` error.
will cause an `ERR_STREAM_PREMATURE_CLOSE` error.
- version: v14.0.0
pr-url: https://github.com/nodejs/node/pull/31509
description: Callback will be invoked on streams which have already
finished before the call to `finished(stream, cb)`.
finished before the call to `finished(stream, cb)`.
-->

* `stream` {Stream} A readable and/or writable stream.
Expand Down Expand Up @@ -1598,9 +1598,9 @@ changes:
- version: v14.0.0
pr-url: https://github.com/nodejs/node/pull/32158
description: `pipeline(..., cb)` will wait for the `'close'` event
before invoking the callback. The implementation tries to detect legacy
streams and only apply this behavior to streams which are expected
to emit `'close'`.
before invoking the callback. The implementation tries to
detect legacy streams and only apply this behavior to streams
which are expected to emit `'close'`.
-->

* `source` {Stream|Iterable|AsyncIterable|Function}
Expand Down