FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

doc: fix stream iterator helpers examples by benjamingr · Pull Request #46897 · nodejs/node · GitHub

/ node Public

doc: fix stream iterator helpers examples - #46897

Merged
nodejs-github-bot merged 1 commit into
mainfrom
benjamingr-improve-stream-docs
Mar 1, 2023
Merged

doc: fix stream iterator helpers examples#46897
nodejs-github-bot merged 1 commit into
mainfrom
benjamingr-improve-stream-docs

Conversation

Copy link
Copy Markdown
Member

Our examples don't run because of a typo and when I asked for feedback on something unrelated I got told that :)

Trying to create a PR from the GitHub UI since I am OOO for a bit. I haven't done a PR from GH in a while so hopefully it's not too much trouble

cc @MoLow @ronag @nodejs/streams

nodejs-github-bot added doc Issues and PRs related to the documentations. stream Issues and PRs related to the stream subsystem. labels Mar 1, 2023
MoLow added the fast-track PRs that do not need to wait for 48 hours to land. label Mar 1, 2023

github-actions Bot commented Mar 1, 2023

Copy link
Copy Markdown
Contributor

Fast-track has been requested by @MoLow. Please 👍 to approve.

benjamingr changed the title docs: fix stream iterator helpers examples doc: fix stream iterator helpers examples Mar 1, 2023
MoLow force-pushed the benjamingr-improve-stream-docs branch from f03fb05 to 04bf7dc Compare March 1, 2023 14:43

MoLow commented Mar 1, 2023

Copy link
Copy Markdown
Member

I have amended the commit message since @benjamingr cannot from his phone

MoLow force-pushed the benjamingr-improve-stream-docs branch from 04bf7dc to d9e47f1 Compare March 1, 2023 14:45

Copy link
Copy Markdown
Member Author

I have amended the commit message since @benjamingr cannot from his phone

Thanks, I enjoy the irony of having a typo of an "s" on a PR that fixes a typo of an "s". Also I have a laptop here just a shitty one (not mine) without Node built ^^

debadree25 added the commit-queue Add this label to land a pull request using GitHub Actions. label Mar 1, 2023
nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Mar 1, 2023
nodejs-github-bot merged commit 8e4fa26 into main Mar 1, 2023
nodejs-github-bot deleted the benjamingr-improve-stream-docs branch March 1, 2023 17:13

Copy link
Copy Markdown
Collaborator

Landed in 8e4fa26

targos pushed a commit that referenced this pull request Mar 13, 2023
PR-URL: #46897
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Robert Nagy <ronagy@icloud.com>
Reviewed-By: Debadree Chatterjee <debadree333@gmail.com>
targos pushed a commit that referenced this pull request Mar 14, 2023
PR-URL: #46897
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Robert Nagy <ronagy@icloud.com>
Reviewed-By: Debadree Chatterjee <debadree333@gmail.com>
danielleadams pushed a commit that referenced this pull request Apr 11, 2023
PR-URL: #46897
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Robert Nagy <ronagy@icloud.com>
Reviewed-By: Debadree Chatterjee <debadree333@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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 the documentations. fast-track PRs that do not need to wait for 48 hours to land. stream Issues and PRs related to the stream subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants


Back | FazBrowse Home | New Git URL