From 24f808fa82444a9395c7b23111ebd40730e01db4 Mon Sep 17 00:00:00 2001 From: John Henderson Date: Tue, 23 Jan 2024 09:55:40 +0000 Subject: [PATCH] Apply further grammar, clarity and spelling fixes to PR tips. Batched suggestions from Ted. Co-authored-by: Ted Thibodeau Jr --- CONTRIBUTING.md | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index da495be..5cb9303 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -72,16 +72,16 @@ rendering of API files* ### Pull request reviewers Reviewers should be added by the pull requester. A "healthy" number of reviewers, usually at least 3 people -other than yourself, is good. +other than the PR author, is good. ### Merging Pull Requests -It is recommended to merge PR using a "Rebase merge", rather than a "Squash and merge". -Merge commits are disabled in order maintain a linear commit history. -"Rebase and merge" is prefered because it maintains the authorship and commit date information which can be important for determining intellectual property contributions. +PRs should be merged using a "Rebase merge", rather than a "Squash and merge". +Merge commits are disabled to maintain a linear commit history. +"Rebase and merge" is preferred because it maintains the authorship and commit date information which can be important for determining intellectual property contributions. ## Edit style guide -The following are style tips when editing the specification. +The following are style tips when editing the specification: - Try to have max of 80 char (do hard wrap, not auto wrap) -- Indent HTML to the proper level using two-spaces per level. +- Indent HTML to the proper level using two spaces per level. - Left justify prose to the first column \ No newline at end of file