diff options
author | Jett Rink <jettrink@chromium.org> | 2019-10-02 11:24:02 -0600 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2021-09-08 00:47:08 +0000 |
commit | 925482a25807aafd3b5e9c3d143c07d19c12b633 (patch) | |
tree | 93f8cfcd526c8034c1188bffdd759c4d05a45ced /docs/code_reviews.md | |
parent | e8e40216636958d1ad06021076fa548d5ab8a58b (diff) | |
download | chrome-ec-925482a25807aafd3b5e9c3d143c07d19c12b633.tar.gz |
doc: add doc for code review rotation
We will use cros-ec-reviewers@google.com account to trigger the
round-robin code reviewer assignment for CLs.
If you are the R line of this CL, then you are on the initial list :)
BRANCH=none
BUG=b:142125160
TEST=builds
Change-Id: I121a8acc984b6b99bb9c1f3638ef431822080f0a
Signed-off-by: Jett Rink <jettrink@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1972607
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
Reviewed-by: Paul Fagerburg <pfagerburg@chromium.org>
Reviewed-by: Keith Short <keithshort@chromium.org>
Reviewed-by: Scott Collyer <scollyer@chromium.org>
Reviewed-by: Eric Yilun Lin <yllin@chromium.org>
Reviewed-by: Diana Z <dzigterman@chromium.org>
(cherry picked from commit c8c2fc0f0790ad8266855f9224aefcc2a447defb)
Signed-off-by: Gwendal Grignou <gwendal@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3140773
Reviewed-by: Aseda Aboagye <aaboagye@chromium.org>
Commit-Queue: Jora Jacobi <jora@google.com>
Owners-Override: Jora Jacobi <jora@google.com>
Diffstat (limited to 'docs/code_reviews.md')
-rw-r--r-- | docs/code_reviews.md | 33 |
1 files changed, 33 insertions, 0 deletions
diff --git a/docs/code_reviews.md b/docs/code_reviews.md new file mode 100644 index 0000000000..df99a231a7 --- /dev/null +++ b/docs/code_reviews.md @@ -0,0 +1,33 @@ +# Code Reviews + +The platform/ec repository makes use of a code review system that tries to +evenly distributed code reviews among available reviewers. + +## How to request a review + +Add `cros-ec-reviewers@google.com` to the reviewer line in Gerrit. A background +job will come around and replace the `cros-ec-reviewers@google.com` address with +the next available reviewer in the EC reviewer rotation. This typically takes on +the order of minutes. + +Optionally, you can click the [FIND OWNERS] button in the UI, and select +`cros-ec-reviewers@google.com` + +## When to use review system + +If you are modifying code in `common/`, `chip/`, or `core/`, feel free to use +the `cros-ec-reviewers@google.com` system. It is **never** a requirement to use +`cros-ec-reviewers@google.com`. You can always request a review from a specific +person. + +## Responsibilities of reviewers + +If the selected reviewer is unfamiliar with code in a CL, then that reviewer +should at least ensure that EC style and paradigms are being followed. Once EC +styles and paradigms are being followed, then the reviewer can give a +1 and add +the appropriate domain expert for that section of code. + +## How can I join the rotation? + +The list of reviewer is at http://google3/chrome/crosinfra/gwsq/ec_reviewers; +just add your name there. |