From 34f3671c09d917e387b36a2b6077f8f282b178e7 Mon Sep 17 00:00:00 2001 From: "J. Ryan Stinnett" Date: Wed, 18 Mar 2020 10:59:33 +0000 Subject: [PATCH] Add notes on review request handling --- docs/review.md | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/docs/review.md b/docs/review.md index 23d8c1c5..e0e297da 100644 --- a/docs/review.md +++ b/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.