From 370cef012e81f11a87fcf7f1384dbbca6cab0c6c Mon Sep 17 00:00:00 2001 From: Arnaud Porterie Date: Fri, 3 Apr 2015 14:04:44 -0700 Subject: [PATCH] Move docs approval section Signed-off-by: Arnaud Porterie --- MAINTAINERS | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index 546e456b0f..2c897902d2 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -193,6 +193,11 @@ for each. # They should ask for any editorial change that makes the documentation more # consistent and easier to understand. # + # Changes and additions to docs must be reviewed and approved (LGTM'd) by a minimum of + # two docs sub-project maintainers. If the docs change originates with a docs + # maintainer, only one additional LGTM is required (since we assume a docs maintainer + # approves of their own PR). + # # Once documentation is approved (see below), a maintainer should make sure to remove this # label and add the next one. @@ -201,11 +206,6 @@ for each. 1-design-review = "raises design concerns" 4-merge = "general case" - # Docs approval - [Rules.review.docs-approval] - # Changes and additions to docs must be reviewed and approved (LGTM'd) by a minimum of two docs sub-project maintainers. - # If the docs change originates with a docs maintainer, only one additional LGTM is required (since we assume a docs maintainer approves of their own PR). - # Merge [Rules.review.states.4-merge]