Revise HowToContribute.md (#5445)
author이한종/On-Device Lab(SR)/Engineer/삼성전자 <hanjoung.lee@samsung.com>
Thu, 20 Jun 2019 04:55:57 +0000 (13:55 +0900)
committer이춘석/On-Device Lab(SR)/Staff Engineer/삼성전자 <chunseok.lee@samsung.com>
Thu, 20 Jun 2019 04:55:57 +0000 (13:55 +0900)
Revise HowToContribute.md with our new reviewing process

Signed-off-by: Hanjoung Lee <hanjoung.lee@samsung.com>
docs/HowToContribute.md

index e626669..a685c8e 100644 (file)
@@ -53,10 +53,16 @@ This section explains the steps to create a pull request (PR).
 
 1. Request review
 
-   Please assign reviewers if you need review from them. Maintainers will honor your review request,
-   and accept your pull request only when all the reviewer approve your pull request. Note that this
-   does **NOT** mean that you should assign reviewers. Maintainers (or reviewers) will review your
-   pull request even without explicit review request.
+   It is recommended to assign reviewers yourself. Maintainers will honor your review request,
+   and accept your pull request only when
+
+   - Approved by 1+ reviewers
+   - 0 rejection(Request Changes)
+   - 0 pending review request
+   - All the reviewers in the list must approve your pull request
+
+   You can add/remove pending review requests in the middle of the review process. Maintainers
+   (or reviewers) could review your pull request even without explicit review request.
 
 1. Update per feedback