Skip to content

Commit

Permalink
doc: use serial comma in pull request doc
Browse files Browse the repository at this point in the history
PR-URL: #43319
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Akhil Marsonya <akhil.marsonya27@gmail.com>
  • Loading branch information
F3n67u authored and danielleadams committed Jun 11, 2022
1 parent 601d933 commit e3798c5
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions doc/contributing/pull-requests.md
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ Node.js has many channels on the
[OpenJS Foundation Slack](https://slack-invite.openjsf.org/). Interesting
channels are:
[#nodejs](https://openjs-foundation.slack.com/archives/CK9Q4MB53) for general
help, questions and discussions.
help, questions, and discussions.
[#nodejs-dev](https://openjs-foundation.slack.com/archives/C019Y2T6STH) for
development of Node.js core specifically.

Expand Down Expand Up @@ -190,7 +190,7 @@ A good commit message should describe what changed and why.

5. If your commit introduces a breaking change (`semver-major`), it should
contain an explanation about the reason of the breaking change, which
situation would trigger the breaking change and what is the exact change.
situation would trigger the breaking change, and what is the exact change.

Sample complete commit message:

Expand Down Expand Up @@ -336,7 +336,7 @@ say so, or contact one of the other contributors in the project and seek their
input. Often such comments are the result of the reviewer having only taken a
short amount of time to review and are not ill-intended. Such issues can often
be resolved with a bit of patience. That said, reviewers should be expected to
be helpful in their feedback, and feedback that is simply vague, dismissive and
be helpful in their feedback, and feedback that is simply vague, dismissive, and
unhelpful is likely safe to ignore.

### Step 10: Landing
Expand Down Expand Up @@ -393,7 +393,7 @@ Focus first on the most significant aspects of the change:
When changes are necessary, _request_ them, do not _demand_ them, and do not
assume that the submitter already knows how to add a test or run a benchmark.

Specific performance optimization techniques, coding styles and conventions
Specific performance optimization techniques, coding styles, and conventions
change over time. The first impression you give to a new contributor never does.

Nits (requests for small changes that are not essential) are fine, but try to
Expand Down

0 comments on commit e3798c5

Please sign in to comment.