From 008da5317608fb2ff3113bdebfc6627b9ddf1881 Mon Sep 17 00:00:00 2001 From: Douwe Maan Date: Tue, 2 Oct 2018 15:13:24 +0000 Subject: Revert "Merge branch 'code_review-assign-maintainers-different-team' into 'master'" This reverts merge request !21644 --- doc/development/code_review.md | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/doc/development/code_review.md b/doc/development/code_review.md index edf0b6f46df..e50e6370c80 100644 --- a/doc/development/code_review.md +++ b/doc/development/code_review.md @@ -27,9 +27,8 @@ There are a few rules to get your merge request accepted: ask or assign any [reviewers][projects] for a first review. 1. If you need some guidance (e.g. it's your first merge request), feel free to ask one of the [Merge request coaches][team]. - 1. It is recommended that you assign a maintainer that is from a different team than your own. - This ensures that all code across GitLab is consistent and can be easily understood by all contributors. - + 1. The reviewer will assign the merge request to a maintainer once the + reviewer is satisfied with the state of the merge request. 1. Keep in mind that maintainers are also going to perform a final code review. The ideal scenario is that the reviewer has already addressed any concerns the maintainer would have found, and the maintainer only has to perform the -- cgit v1.2.1