From b3e1d851aaf10e003ad1889ef56c080c76beca56 Mon Sep 17 00:00:00 2001 From: Manoj M J <mmj@gitlab.com> Date: Mon, 22 Aug 2022 19:27:05 +0000 Subject: [PATCH] Update docs around Review limits --- doc/development/code_review.md | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/doc/development/code_review.md b/doc/development/code_review.md index ec3e8220845eb..48d3a0aeb6549 100644 --- a/doc/development/code_review.md +++ b/doc/development/code_review.md @@ -100,6 +100,12 @@ page, with these behaviors: - 3ï¸âƒ£ - `:three:` - 4ï¸âƒ£ - `:four:` - 5ï¸âƒ£ - `:five:` + + Review requests for merge requests that do not target the default branch of any + project under the [security group](https://gitlab.com/gitlab-org/security/) are + not counted. These MRs are usually backports, and maintainers or reviewers usually + do not need much time reviewing them. + - Team members whose Slack or [GitLab status](../user/profile/index.md#set-your-current-status) emoji is 🔵 `:large_blue_circle:` are more likely to be picked. This applies to both reviewers and trainee maintainers. - Reviewers with 🔵 `:large_blue_circle:` are two times as likely to be picked as other reviewers. -- GitLab