Skip to content

Commit fbdc22a

Browse files
authored
Merge pull request #2088 from yaahc/review-process-overview
Add review process overview to review-policy.md
2 parents ef2c2f2 + 81ed7c1 commit fbdc22a

File tree

1 file changed

+8
-0
lines changed

1 file changed

+8
-0
lines changed

docs/review-policy.md

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,13 @@
11
Team members are given permission to merge changes from other contributors in the <https://github.com/rust-lang/reference> repository. There are different guidelines for reviewing based on the kind of changes being made:
22

3+
## Review principles
4+
5+
Reviewers and authors should focus on a few key principles during the review process:
6+
7+
* **Understandability**: Language within the Reference should be understandable to most members of the Project. Contributions should assumes that readers are familiar with the rest of the content of the Reference, but, wherever possible, sections should facilitate that understanding by linking to related content.
8+
* **Defensibility**: When the lang-docs team merges a change to the Reference, they are agreeing to take responsibility for it going forward. Team members need to feel confident defending and explaining the correctness of content within the Reference. Whenever possible, changes to the Reference should back up any claims with concise examples to verify correctness.
9+
* **Voice**: Authors are not expected to have competence as a specification writer when drafting new contributions to the Reference. So long as claims are understandable and defensible, it is fine for PRs to be written in a casual tone or with the voice of the author instead of the voice of the Reference. Team members will bring editorial experience as part of their reviews and will revise the phrasing, organization, style, etc. to fit the Reference before merging if necessary.
10+
311
## Policy changes
412

513
- Significant changes to the policy of how the team operates, such as changes to this document, should have agreement of the team without any blocking objections.

0 commit comments

Comments
 (0)