Skip to content

doc: fix grammar and punctuation in cli.md - #65237

Closed
Rawal27 wants to merge 5 commits into
nodejs:mainfrom
Rawal27:doc-fix-cli-documentation
Closed

doc: fix grammar and punctuation in cli.md#65237
Rawal27 wants to merge 5 commits into
nodejs:mainfrom
Rawal27:doc-fix-cli-documentation

Conversation

@Rawal27

@Rawal27 Rawal27 commented Aug 12, 2026

Copy link
Copy Markdown

Description

Improve the grammar, spelling, and punctuation in the command-line API documentation.

Changes

  • Correct grammatical issues in descriptions of command-line options.
  • Fix incorrect articles, such as an native to a native and a implicit to an implicit.
  • Correct subject–verb agreement issues.
  • Correct spelling errors such as serializablility to serializability.
  • Fix punctuation and hyphenation issues.
  • Correct terminology such as out-of-bound to out-of-bounds.
  • Improve the clarity of a few grammatically unclear sentences.

Changes are documentation-only. No functional changes.

Signed-off-by: Rawal27 <obviouslykamal@gmail.com>
@nodejs-github-bot nodejs-github-bot added cli Issues and PRs related to the Node.js command line interface. doc Issues and PRs related to the documentations. labels Aug 12, 2026
Signed-off-by: Rawal27 <obviouslykamal@gmail.com>
Comment thread doc/api/cli.md Outdated
MikeMcC399

This comment was marked as resolved.

Rawal27 and others added 3 commits August 12, 2026 14:16
Signed-off-by: Rawal27 <obviouslykamal@gmail.com>
Co-authored-by: Mike McCready <66998419+MikeMcC399@users.noreply.github.com>
Signed-off-by: Rawal27 <obviouslykamal@gmail.com>

@MikeMcC399 MikeMcC399 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@MikeMcC399 MikeMcC399 added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. labels Aug 12, 2026
@MikeMcC399 MikeMcC399 added the commit-queue Add this label to land a pull request using GitHub Actions. label Aug 12, 2026
@aduh95

aduh95 commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Please stop opening PRs faster they can be reviewed. If you do not stop, you will get blocked from the organisation.

@aduh95 aduh95 closed this Aug 13, 2026
@Rawal27

Rawal27 commented Aug 13, 2026

Copy link
Copy Markdown
Author

@aduh95 Can you clarify this? I mean, these are genuine doc-level changes that I have proposed. Also, a contributor can contribute to as many as 30 pull requests at a time. Closing them without reason seems unfair to me.

This one was already approved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. cli Issues and PRs related to the Node.js command line interface. commit-queue Add this label to land a pull request using GitHub Actions. commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. doc Issues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants