Review doc updated

Development
1

The review process documentation has been updated to be coherent with what the robot expect.
https://gitlab.kitware.com/paraview/paraview/-/blob/master/Documentation/dev/git/develop.md#human-reviews

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 +1s do not stack.
2 Likes