Skip to content
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.

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

doc: fix stream iterator helpers examples #46897

Merged
merged 1 commit into from
Mar 1, 2023

Conversation

benjamingr
Copy link
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 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 MoLow added the fast-track PRs that do not need to wait for 48 hours to land. label Mar 1, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Mar 1, 2023

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

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

MoLow commented Mar 1, 2023

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

@MoLow MoLow force-pushed the benjamingr-improve-stream-docs branch from 04bf7dc to d9e47f1 Compare March 1, 2023 14:45
@benjamingr
Copy link
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 debadree25 added the commit-queue Add this label to land a pull request using GitHub Actions. label Mar 1, 2023
@nodejs-github-bot 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 nodejs-github-bot merged commit 8e4fa26 into main Mar 1, 2023
@nodejs-github-bot nodejs-github-bot deleted the benjamingr-improve-stream-docs branch March 1, 2023 17:13
@nodejs-github-bot
Copy link
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>
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