Skip to content

Commit

Permalink
doc: use serial comma in pull request doc
Browse files Browse the repository at this point in the history
  • Loading branch information
F3n67u committed Jun 5, 2022
1 parent 45d7ca9 commit 6d06eef
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 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 @@ -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 6d06eef

Please sign in to comment.