Skip to content

Commit 34bc5da

Browse files
committed
Update pull_request.md with 2 reviewers and a new reviewer team
1 parent a521824 commit 34bc5da

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

docs/pull_request.md

+6-5
Original file line numberDiff line numberDiff line change
@@ -32,14 +32,15 @@ Also, draft PR should not stay indefinitely in this state. It may be removed if
3232

3333
##### PR Review Assignment
3434

35-
We use automatic assignment for PR reviews. A PR is automatically routed by GitHub to a team member using the round robin algorithm. The process is the following:
35+
We use automatic assignment for PR reviews. A PR is automatically routed by GitHub to 2 team members using the round robin algorithm. The process is the following:
3636

37-
- The PR creator assigns the [element-android](https://github.com./orgs/vector-im/teams/element-android) team as a reviewer. They can skip this process and assign directly a specific member if they think they should take a look at it.
38-
- GitHub automatically assigns one reviewer. If the chosen reviewer is not available (holiday, etc.), remove them and set again the team, GitHub will select another reviewer.
39-
- The reviewer gets a notification to make the review: they review the code following the good practice (see the rest of this document).
37+
- The PR creator can assign specific people if they have another Android developer in their team or they think a specific reviewer should take a look at the PR.
38+
- If there are missing reviewers, the PR creator assigns the [element-android-reviewers](https://github.com./orgs/vector-im/teams/element-android-reviewers) team as a reviewer.
39+
- GitHub automatically assigns other reviewers. If one of the chosen reviewers is not available (holiday, etc.), remove them and set again the team, GitHub will select another reviewer.
40+
- Reviewers get a notification to make the review: they review the code following the good practice (see the rest of this document).
4041
- After making their own review, if they feel not confident enough, they can ask another person for a full review, or they can tag someone within a PR comment to check specific lines.
4142

42-
For PRs coming from the community, the issue wrangler can assign either the team [element-android](https://github.com./orgs/vector-im/teams/element-android) or any member directly.
43+
For PRs coming from the community, the issue wrangler can assign either the team [element-android-reviewers](https://github.com./orgs/vector-im/teams/element-android-reviewers) or any members directly.
4344

4445
##### PR review time
4546

0 commit comments

Comments
 (0)