diff options
Diffstat (limited to 'docs/code_reviews.md')
-rw-r--r-- | docs/code_reviews.md | 54 |
1 files changed, 0 insertions, 54 deletions
diff --git a/docs/code_reviews.md b/docs/code_reviews.md deleted file mode 100644 index a8e77c3fd8..0000000000 --- a/docs/code_reviews.md +++ /dev/null @@ -1,54 +0,0 @@ -# Code Reviews - -The `platform/ec` repository makes use of a code review system that tries to -evenly distribute code reviews among available reviewers. - -[TOC] - -## 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. - -Reviewers should try to give an initial response within 1 business day of -receiving a review request. Thereafter, they should try to respond to new -comments by the author within 1 business day. - -## Review guidelines - -Authors and reviewers should follow the Chrome OS firmware review -[guidelines][2] while publishing and reviewing code. - -## How can I join the rotation? - -Add your name to the [list of reviewers][1]. - -## Reference - -* [Chrome OS firmware review guidelines][2] -* [Coreboot Gerrit Guidelines][3] -* [Google small CL guidelines][5] - -[1]: http://google3/chrome/crosinfra/gwsq/ec_reviewers -[2]: http://chromium.googlesource.com/chromiumos/docs/+/HEAD/firmware_code_reviews.md -[3]: https://doc.coreboot.org/getting_started/gerrit_guidelines.html -[5]: https://google.github.io/eng-practices/review/developer/small-cls.html |