From 3bcd2006b56ad57caa7e8723c3269cbd50af1f4a Mon Sep 17 00:00:00 2001 From: Fred Lifton Date: Wed, 11 Mar 2015 17:38:35 -0700 Subject: [PATCH] Adds Fred to list of people and clarifies that docs approval requires only two LGTMS (not a "majority" as it was before Mary and Steve were added). Docker-DCO-1.1-Signed-off-by: Fred Lifton (github: fredlf) --- MAINTAINERS | 12 +++++++++++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index fba5445b7d..04951bf459 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -193,13 +193,18 @@ for each. # They should ask for any editorial change that makes the documentation more # consistent and easier to understand. # - # Once documentation is approved, a maintainer should make sure to remove this + # Once documentation is approved (see below), a maintainer should make sure to remove this # label and add the next one. close = "" 2-code-review = "requires more code changes" 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] @@ -535,6 +540,11 @@ made through a pull request. Email = "estesp@linux.vnet.ibm.com" GitHub = "estesp" + [people.fredlf] + Name = "Fred Lifton" + Email = "fred.lifton@docker.com" + GitHub = "fredlf" + [people.icecrime] Name = "Arnaud Porterie" Email = "arnaud@docker.com"