Skip to content

doc: add notes about non-conforming streams #29895

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 8 commits into from
Closed

doc: add notes about non-conforming streams #29895

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
7ce4453
doc: add notes about non-confirming streams
ronag Oct 9, 2019
e2207eb
fixup
ronag Oct 20, 2019
1ac15d1
fixup
ronag Oct 20, 2019
fc5029d
Update doc/api/stream.md
ronag Nov 19, 2019
fdba619
fixup: provide examples of methods
ronag Nov 24, 2019
6f4b705
Apply suggestions from code review
ronag Nov 25, 2019
50a3d5a
fixup: line wrap
ronag Nov 25, 2019
687ead7
fixup: events plural
ronag Nov 25, 2019
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
Next Next commit
doc: add notes about non-confirming streams
  • Loading branch information
@ronag
ronag committed Oct 9, 2019
commit 7ce445303706a79b58a0c7677ae9577ca0f0f4fa
6 changes: 6 additions & 0 deletions doc/api/stream.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1667,6 +1667,12 @@ of a stream that are intended for use by consumers (as described in the
[API for Stream Consumers][] section). Doing so may lead to adverse side effects
in application code consuming the stream.

It is highly discouraged to override any public method. As well as emitting
Comment thread
ronag marked this conversation as resolved.
Outdated
`'error'` events manually through `.emit(err)` instead of using API provided
callbacks or `.destroy(err)`. Doing so can break current and future stream
invariants leading to behaviour and/or compatibility issues with other streams,
Comment thread
ronag marked this conversation as resolved.
Outdated
stream utilities and user expectations.
Comment thread
ronag marked this conversation as resolved.
Outdated
Comment thread
ronag marked this conversation as resolved.
Outdated

### Simplified Construction
<!-- YAML
added: v1.2.0
Expand Down