Skip to content

Add specification of what area to review for multiple areas

Doug Stull requested to merge docs-specify-which-area-to-review into master

What does this MR do and why?

Provides guidance on requesting a review when the MR touches multiple areas and thus requires multiple reviews.

Scenario this solves:

  1. A MR has database, backend and frontend review concerns. (or any combination of areas)
  2. A person requests me to review and danger is not followed or out of date.
  3. I have no real idea which area I'm being asked to review since I am a reviewer of multiple areas.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports