# GitLab code review process and documentation updated

**URL:** https://discourse.vtk.org/t/gitlab-code-review-process-and-documentation-updated/10271
**Category:** Development
**Created:** [December 20, 2022, 9:47am UTC](https://discourse.vtk.org/t/gitlab-code-review-process-and-documentation-updated/10271 "2022-12-20T09:47:30Z")
**Posts on this page:** 1
**Page:** 1

<div class="post-metadata">

### Author: ![mwestphal](https://discourse.vtk.org/user_avatar/discourse.vtk.org/mwestphal/32/19_2.png) [@mwestphal](https://discourse.vtk.org/u/mwestphal)
#### Post date: [December 20, 2022, 9:47am UTC](https://discourse.vtk.org/t/gitlab-code-review-process-and-documentation-updated/10271/1 "2022-12-20T09:47:30Z")

</div>

The review process documentation has been updated to be coherent with what the robot expect.  
[https://gitlab.kitware.com/vtk/vtk/-/blob/master/Documentation/dev/git/develop.md#leading-line](https://gitlab.kitware.com/vtk/vtk/-/blob/master/Documentation/dev/git/develop.md#leading-line)

The main take away is.

- `+1` means acknowleded by, aka LGTM, aka a quick look into it.
- `+2` means reviewed by, aka deep look, line by line, review
- `+3` means locally tested (and also deep look review)
- In order to be considered mergeable, any MR needs at least one `+2`, and `+1`s do not stack.