We need to clearly document how repository review/commit permissions will be setup in Phabricator both for migrated repositories and any new repositories going forward.
This could be accomplished through acl_* projects
Description
Description
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Declined | None | T207 Update Code Review related documentation on wiki pages from Gerrit to Differential | |||
| Declined | None | T130786 Document permissions standards for code-reviewers/committers |
Event Timeline
Comment Actions
If we want to implement a +2 group, we probably should wait for https://secure.phabricator.com/T8887, it allows to use owners for that. (Maybe pay them for make the prio higher), this saves herald rules, and makes it more clear.
Comment Actions
Without comments from an upstream dev on that task I have no idea how likely it is to be approved as a thing they would want doing :)
Comment Actions
It's fixed! The fixing commit: https://secure.phabricator.com/rP332d787dc8f43389fd531d027bbfbe50d06df5d7