Add notes on review request handling

pull/12730/head
J. Ryan Stinnett 2 years ago
parent 3989f41985
commit 34f3671c09
  1. 5
      docs/review.md

@ -40,6 +40,11 @@ When reviewing code, here are some things we look for and also things we avoid:
### Workflow
* Authors should request review from riot-web team by default (if someone on the
team is clearly the expert in an area, a direct review request to them may be
more appropriate)
* Reviewers should remove the team review request and request review from
themselves when starting a review to avoid double review
* Authors should link to other layers of their PR in their PR before requesting
review. Reviewers might be coming from different places and could miss other
required PRs.

Loading…
Cancel
Save