Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

ronijames008
Copy link
Contributor

doc: add comma after "blocks" for clarity in child_process.md

This PR improves clarity in doc/api/child_process.md by adding a comma after the word "blocks" in the sentence:

... the subprocess blocks, waiting for the pipe buffer to accept more data. ...

The change improves readability and ensures proper grammatical flow. No functional code is affected.

No functional code is affected.

@nodejs-github-bot nodejs-github-bot added child_process Issues and PRs related to the child_process subsystem. doc Issues and PRs related to the documentations. labels Jun 27, 2025
@aduh95
Copy link
Contributor

aduh95 commented Jun 27, 2025

The git commit message does not follow our guidelines, an example of a valid commit message would be doc: add missing comma in `child_process.md`

Adds a comma after "blocks" in a sentence explaining how subprocesses
may pause when the pipe buffer is full. This improves readability and
clarifies the meaning of the sentence, preventing misinterpretation of
"blocks waiting" as a compound phrase.

No functional code is affected.
@ronijames008 ronijames008 force-pushed the docs/fix-grammar-blocks branch from ef49538 to 0682f79 Compare June 28, 2025 11:37
@ronijames008 ronijames008 changed the title doc: child_process: add comma after “blocks” doc: add missing comma in child_process.md Jun 28, 2025
@ronijames008 ronijames008 force-pushed the docs/fix-grammar-blocks branch from 0682f79 to ce79106 Compare June 28, 2025 11:53
@jasnell jasnell added the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 29, 2025
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 29, 2025
@nodejs-github-bot nodejs-github-bot merged commit bab36c3 into nodejs:main Jun 29, 2025
18 checks passed
@nodejs-github-bot
Copy link
Collaborator

Landed in bab36c3

targos pushed a commit that referenced this pull request Jul 3, 2025
Adds a comma after "blocks" in a sentence explaining how subprocesses
may pause when the pipe buffer is full. This improves readability and
clarifies the meaning of the sentence, preventing misinterpretation of
"blocks waiting" as a compound phrase.

No functional code is affected.

PR-URL: #58862
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Ulises Gascón <ulisesgascongonzalez@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
aduh95 pushed a commit that referenced this pull request Jul 21, 2025
Adds a comma after "blocks" in a sentence explaining how subprocesses
may pause when the pipe buffer is full. This improves readability and
clarifies the meaning of the sentence, preventing misinterpretation of
"blocks waiting" as a compound phrase.

No functional code is affected.

PR-URL: #58862
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Ulises Gascón <ulisesgascongonzalez@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
aduh95 pushed a commit that referenced this pull request Jul 24, 2025
Adds a comma after "blocks" in a sentence explaining how subprocesses
may pause when the pipe buffer is full. This improves readability and
clarifies the meaning of the sentence, preventing misinterpretation of
"blocks waiting" as a compound phrase.

No functional code is affected.

PR-URL: #58862
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Ulises Gascón <ulisesgascongonzalez@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

child_process Issues and PRs related to the child_process subsystem. doc Issues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.