opendev/infra-manual
Code Issues Proposed changes

Update core.rst for two +2 policy

Browse Source 
Two +2s is a policy by some projects that is not enforced, update the
approval section for this.

Change-Id: Ibf569f17444d4d9512d53ef90f2a075f6a093125
This commit is contained in:
Andreas Jaeger 2020-03-04 21:23:05 +01:00
parent d4b2082152
commit f6ac75a535
1 changed files with 9 additions and 4 deletions

13
doc/source/core.rst

@ -25,10 +25,15 @@ person that can remove it again.
Approval
========
Once a patch has two +2 votes and neither any -2 votes or a Work in
Progress tag set, it is ready to be merged. Any core reviewer can
approve such a patch. A core reviewer can give a +2 and approval in
the same review.
Once a patch has at least one +2 vote and neither any -2 votes or a
Work in Progress tag set, it is ready to be merged. Any core reviewer
can approve such a patch. A core reviewer can give a +2 and approval
in the same review.
Note that some projects hosted in OpenDev have a policy of requiring
*two* +2 votes instead of one. The OpenDev infrastructure enforces
only a single +2 vote, the rest is policy.
Work In Progress
================