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
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
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 |
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.
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 :)
It's fixed! The fixing commit: https://secure.phabricator.com/rP332d787dc8f43389fd531d027bbfbe50d06df5d7