From f6ac75a5357487319b2141ecedd1e204fb045736 Mon Sep 17 00:00:00 2001 From: Andreas Jaeger Date: Wed, 4 Mar 2020 21:23:05 +0100 Subject: [PATCH] Update core.rst for two +2 policy Two +2s is a policy by some projects that is not enforced, update the approval section for this. Change-Id: Ibf569f17444d4d9512d53ef90f2a075f6a093125 --- doc/source/core.rst | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/doc/source/core.rst b/doc/source/core.rst index 28bf1dc..e36963f 100644 --- a/doc/source/core.rst +++ b/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 ================