Skip to content

doc: add missing comma in child_process.md #58862

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ronijames008
Copy link

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
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.

3 participants