Skip to content

stream: improve comments regarding end() errors. - #32839

Closed
ronag wants to merge 1 commit into
nodejs:masterfrom
nxtedition:stream-update-comments
Closed

ronag wants to merge 1 commit into
nodejs:masterfrom
nxtedition:stream-update-comments

Conversation

@ronag

@ronag ronag commented Apr 14, 2020

Copy link
Copy Markdown
Member

Cleans up comments and TODOs and tries to provide a
more detail description in regards to error behavior
of end().

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

Cleans up comments and TODOs and tries to provide a
more detail description in regards to error behavior
of end().
@ronag ronag added the stream Issues and PRs related to Node.js streams. label Apr 14, 2020
@ronag
ronag requested review from BridgeAR, lpinca and mcollina April 14, 2020 06:28

@mcollina mcollina left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@ronag ronag mentioned this pull request Apr 14, 2020
4 tasks done
@ronag ronag added dont-land-on-v10.x author ready PRs with CI started, the required approvals, and no outstanding review comments. labels Apr 14, 2020
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

nodejs-github-bot commented Apr 14, 2020 •

Copy link
Copy Markdown
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/30718/ (:heavy_check_mark:)

ronag added a commit that referenced this pull request Apr 16, 2020
Cleans up comments and TODOs and tries to provide a
more detail description in regards to error behavior
of end().

PR-URL: #32839
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@ronag

ronag commented Apr 16, 2020

Copy link
Copy Markdown
Member Author

Landed in 4a6a5c3

@ronag ronag closed this Apr 16, 2020
MylesBorins pushed a commit that referenced this pull request Apr 17, 2020
Cleans up comments and TODOs and tries to provide a
more detail description in regards to error behavior
of end().

PR-URL: #32839
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs with CI started, the required approvals, and no outstanding review comments. stream Issues and PRs related to Node.js streams.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants