Skip to content

doc: fix paragraph order in stream.md - #19501

Closed
vsemozhetbyt wants to merge 1 commit into
nodejs:masterfrom
vsemozhetbyt:doc-stream-order
Closed

vsemozhetbyt wants to merge 1 commit into
nodejs:masterfrom
vsemozhetbyt:doc-stream-order

Conversation

@vsemozhetbyt

Copy link
Copy Markdown
Contributor
Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • documentation is changed or added
  • commit message follows commit guidelines

Place writable.destroy(), readable.destroy(), and readable.read() in the proper ABC order.

Place `writable.destroy()`, `readable.destroy()`,
and `readable.read()` in the proper ABC order.
@vsemozhetbyt vsemozhetbyt added the fast-track PRs proposed for a shorter-than-standard waiting period before landing. label Mar 21, 2018
@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to Node.js documentation. stream Issues and PRs related to Node.js streams. labels Mar 21, 2018
@vsemozhetbyt

vsemozhetbyt commented Mar 21, 2018 •

Copy link
Copy Markdown
Contributor Author

@Trott Trott added the author ready PRs with CI started, the required approvals, and no outstanding review comments. label Mar 21, 2018
vsemozhetbyt added a commit that referenced this pull request Mar 21, 2018
Place `writable.destroy()`, `readable.destroy()`,
and `readable.read()` in the proper ABC order.

PR-URL: #19501
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
@vsemozhetbyt

Copy link
Copy Markdown
Contributor Author

Landed in 3662934

@vsemozhetbyt
vsemozhetbyt deleted the doc-stream-order branch March 21, 2018 02:54
@tniessen tniessen removed the author ready PRs with CI started, the required approvals, and no outstanding review comments. label Mar 24, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Issues and PRs related to Node.js documentation. fast-track PRs proposed for a shorter-than-standard waiting period before landing. stream Issues and PRs related to Node.js streams.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants