Skip to content

Commit 31d8863

Browse files
authored
Add some autoassigner notes (#6154)
1 parent c713279 commit 31d8863

File tree

1 file changed

+13
-0
lines changed

1 file changed

+13
-0
lines changed

docs/project/code_review.md

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@ SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
2828
- [What should be covered by a review?](#what-should-be-covered-by-a-review)
2929
- [Writing review comments](#writing-review-comments)
3030
- [Approving the change](#approving-the-change)
31+
- [Pausing reviews and reassigning PRs](#pausing-reviews-and-reassigning-prs)
3132
- [Merging pull requests](#merging-pull-requests)
3233
- [Merge commit descriptions](#merge-commit-descriptions)
3334
- [Resolving an impasse or conflict](#resolving-an-impasse-or-conflict)
@@ -440,6 +441,18 @@ author can always come back to you if they have questions, and we can always
440441
revert changes if the resolution for some reason diverges wildly from your
441442
expectations.
442443

444+
### Pausing reviews and reassigning PRs
445+
446+
When temporarily unavailable to review, for example due to a vacation, reviewers
447+
can either mark themselves as
448+
[busy in GitHub](https://docs.github.com/en/account-and-profile/tutorials/personalize-your-profile#setting-a-status),
449+
or ask an admin to stop assignment (using
450+
["Never assign to certain team members"](https://docs.github.com/en/organizations/organizing-members-into-teams/managing-code-review-settings-for-your-team#configuring-auto-assignment)).
451+
452+
A PR's reviewer can also be changed to a team, such as
453+
`carbon-language/toolchain-reviewers`, and it should be automatically
454+
reassigned. This can be done by anyone, not just a reviewer.
455+
443456
## Merging pull requests
444457

445458
Pull requests are ready to be merged when reviewers have indicated they're happy

0 commit comments

Comments
 (0)