Jhernandez has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/370772 )

Change subject: Chore: update Gerrit permissions
......................................................................


Chore: update Gerrit permissions

- Keep Marvin members as owner for refs/* but remove all other
  permissions until they're necessary. The intent was to keep
  permissions minimal although an ideal configuration is unknown.

- For master, enable JenkinsBot, l10n-bot, and Marvin members to label
  patchsets verified and submit them. It is hoped that this will allow
  Marvin members to merge code! Label verification should probably be
  revoked from Marvin members once JenkinsBot is configured.

References:
- https://gerrit.wikimedia.org/r/#/admin/projects/marvin,access
- https://gerrit.wikimedia.org/r/#/admin/projects/apps/android/wikipedia,access
- 
https://gerrit.wikimedia.org/r/#/admin/projects/mediawiki/extensions/MobileFrontend,access
- https://gerrit.wikimedia.org/r/#/admin/projects/mediawiki/skins/Vector,access

Change-Id: I89d5028f324a97a442843bd928cce51051dc3aca
---
M groups
M project.config
2 files changed, 8 insertions(+), 5 deletions(-)

Approvals:
  Jhernandez: Verified; Looks good to me, approved



diff --git a/groups b/groups
index adab094..96f0bff 100644
--- a/groups
+++ b/groups
@@ -1,4 +1,5 @@
 # UUID                                         Group Name
 #
 2bc47fcadf4e44ec9a1a73bcfa06232554f47ce2       JenkinsBot
+cc37d98e3a4301744a0c0a9249173ae170696072       l10n-bot
 cf8dbbd6366e501ba9bdd991e0b4c9614079af74       marvin
diff --git a/project.config b/project.config
index 6a897bb..3b20ffc 100644
--- a/project.config
+++ b/project.config
@@ -6,10 +6,12 @@
        mergeContent = true
 [access "refs/*"]
        owner = group marvin
-       submit = group JenkinsBot
-       forgeAuthor = group marvin
-       forgeCommitter = group marvin
-       create = group marvin
-       push = +force group marvin
 [project]
        description = The new Wikipedia mobile website. 
https://phabricator.wikimedia.org/diffusion/MARV/browse/master/
+[access "refs/heads/master"]
+       label-Verified = -1..+2 group JenkinsBot
+       label-Verified = -1..+2 group l10n-bot
+       label-Verified = -1..+2 group marvin
+       submit = group JenkinsBot
+       submit = group l10n-bot
+       submit = group marvin

-- 
To view, visit https://gerrit.wikimedia.org/r/370772
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I89d5028f324a97a442843bd928cce51051dc3aca
Gerrit-PatchSet: 1
Gerrit-Project: marvin
Gerrit-Branch: refs/meta/config
Gerrit-Owner: Niedzielski <sniedziel...@wikimedia.org>
Gerrit-Reviewer: Jhernandez <jhernan...@wikimedia.org>
Gerrit-Reviewer: Paladox <thomasmulhall...@yahoo.com>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to