Reviewer Roulette: Note Quality maintainer assignment is optional for ~test and ~QA categories
Reviewer Roulette includes a reviewer and maintainer for test and QA categories, and it says:
A merge request is normally reviewed by both a reviewer and a maintainer in its primary category (e.g. frontend or backend), and by a maintainer in all other categories.
However, that's not the case for test and QA -- we are typically satisfied with one review and it doesn't have to be a maintainer.
Question: Are we still satisfied with allowing only a non-maintainer review for QA and test categories? (I think so)
If the answer is "Yes", then the proposal is to:
-
Update Reviewer Roulette to note that a review by Quality maintainer is optional for test and QA categories. One review is sufficient, and that reviewer doesn't have to be a maintainer.
This is complimentary to gitlab-org/gitlab!32037 (merged)
Edited by Mark Lapierre