diff options
author | Marcia Ramos <virtua.creative@gmail.com> | 2019-02-21 14:42:21 +0300 |
---|---|---|
committer | Marcia Ramos <virtua.creative@gmail.com> | 2019-02-21 15:07:30 +0300 |
commit | 6349a18588b58c57a4095a1364987b8bf4766612 (patch) | |
tree | f519c7b77891831a6286554ffd03f17ecf44f62a /doc | |
parent | 161851de3632c772248b7f2f0d5614b49bd06a21 (diff) |
Update code review guidelines
Diffstat (limited to 'doc')
-rw-r--r-- | doc/development/code_review.md | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/doc/development/code_review.md b/doc/development/code_review.md index 1b591c7c322..8de02d273ec 100644 --- a/doc/development/code_review.md +++ b/doc/development/code_review.md @@ -203,7 +203,10 @@ first time. - Extract unrelated changes and refactorings into future merge requests/issues. - Seek to understand the reviewer's perspective. - Try to respond to every comment. -- Let the reviewer select the "Resolve discussion" buttons. +- The merge request author resolves only the discussions they have fully + addressed. If there's an open reply, an open discussion, a suggestion, + a question, or anything else, the discussion must be left to be resolved + by the reviewer. - Push commits based on earlier rounds of feedback as isolated commits to the branch. Do not squash until the branch is ready to merge. Reviewers should be able to read individual updates based on their earlier feedback. |