Skip to content

Commit 1957b8a

Browse files
authored
Merge pull request #51 from ehuss/review-policy
Add a review policy
2 parents 76139cc + c0fdc7f commit 1957b8a

File tree

1 file changed

+38
-0
lines changed

1 file changed

+38
-0
lines changed

docs/team.md

Lines changed: 38 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,38 @@
1+
# Spec Team
2+
3+
## Mission
4+
5+
## Team membership
6+
7+
### Membership expectations
8+
9+
#### Review policy
10+
11+
Team members are given permission to merge changes from other contributors in the <https://github.com/rust-lang/spec> repository. There are different guidelines for reviewing based on the kind of changes being made:
12+
13+
- Policy changes
14+
- 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.
15+
- Minor changes to something like the style enforcement can be made with the review of a team member, as long as there is high confidence that it is unlikely any team member would object (for example, codifying a guideline that is already in practice), and that the change can be easily reversed.
16+
- Meaningful content addition or changes
17+
- When adding or changing content in the spec, the reviewer should consult with appropriate experts to validate the changes. This may not be required if the reviewer has high confidence that the changes are correct, and consider themselves well-versed enough in the topic to understand it, or the relevant experts are the author or have been heavily involved in the process. It is up to the reviewer to use their best judgement when to consult.
18+
- Content should always follow the guidelines from the [authoring guide].
19+
- Minor content changes
20+
- Minor content changes, such as small cleanups or wording fixes, can be made with the review from a team member without further consultation.
21+
- Tooling changes
22+
- Minor changes to the tooling may be made with a review from a team member. This includes bug fixes, minor additions that are unlikely to have objections, and additions that have already been discussed.
23+
- Major changes, such as a change in how content is authored, or major changes to how the tooling works should be approved by the team without blocking objections.
24+
25+
<!-- TODO -->
26+
This policy does not yet cover the process for getting final approval from the relevant teams.
27+
28+
[authoring guide]: authoring.md
29+
30+
### Meetings
31+
32+
### Becoming a member
33+
34+
## Team resources
35+
36+
## Decision process
37+
38+
## Contact

0 commit comments

Comments
 (0)