Commit 8d280f95 authored by Sean McGivern's avatar Sean McGivern

Merge branch 'cablett-code-review' into 'master'

Add domain expert guidance to code review docs

See merge request gitlab-org/gitlab!51846
parents 3e67e371 754fefb3
......@@ -24,6 +24,7 @@ uncovered edge cases.
The default approach is to choose a reviewer from your group or team for the first review.
This is only a recommendation and the reviewer may be from a different team.
However, it is recommended to pick someone who is a [domain expert](#domain-experts).
If your merge request touches more than one domain (for example, Dynamic Analysis and GraphQL), ask for reviews from an expert from each domain.
You can read more about the importance of involving reviewer(s) in the section on the responsibility of the author below.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment