Skip to content

Improve the code review section in MR templates.

Olivier Gonzalez requested to merge 46297_improve_code_review_section_in_mr_templates into master

What does this MR do?

Improve the code review section in MR templates.

Important The default MR template is not a template file within the repo but a project setting so this have to be updated by a maintainer (for both CE and EE):

have:

- Conform by the [code review guidelines](https://docs.gitlab.com/ee/development/code_review.html)
  - [ ] Has been reviewed by a UX team member
  - [ ] Has been reviewed by a Frontend maintainer
  - [ ] Has been reviewed by a Backend maintainer
  - [ ] Has been reviewed by a Database specialist

instead of:

- Review
  - [ ] Has been reviewed by UX
  - [ ] Has been reviewed by Frontend
  - [ ] Has been reviewed by Backend
  - [ ] Has been reviewed by Database

Why was this MR needed?

To help developers finding the right process when having their MR reviewed.

What are the relevant issue numbers?

#46297 (closed)

Merge request reports