Phabricator 有没有办法更改 "Differential" 中的默认 "Accept" 策略以要求接受 "all" 评论,而不是 "some"?
Phabricator Is there a way to change default "Accept" policy in "Differential" to require "all" reviews to accept, rather than "some"?
根据 'Differential' 文档:
https://secure.phabricator.com/book/phabricator/article/differential/
Once everything is satisfied, some reviewer accepts the change and the author pushes it to the upstream.
这意味着任何审阅者都可以将总体审阅状态标记为 'Accepted',即使某些审阅者可能有未解决的问题。
有没有办法更改它,以便在 所有 位评论者接受时将评论标记为 "Accept"ed?
even though some reviewers may have open questions
这个有点问题。从根本上说,如果您有一位工程师 忽略 审查时的代码反馈并无论如何推送代码,那么应该单独与该工程师交谈。 Phabricator 中灵活的代码审查的要点是大多数人在大多数时间都可以被信任,并且通过该产品可以提高工作流程速度并保持畅通无阻(和工程师工作)。
如果您不信任您的工程师,像 Herald(添加阻止审阅者)和 Owners(代码所有权包)这样的工具仍然可以实现您可能想要的大部分功能。
https://secure.phabricator.com/T731是上游的任务跟踪"all must accept"类型规则。
根据 'Differential' 文档: https://secure.phabricator.com/book/phabricator/article/differential/
Once everything is satisfied, some reviewer accepts the change and the author pushes it to the upstream.
这意味着任何审阅者都可以将总体审阅状态标记为 'Accepted',即使某些审阅者可能有未解决的问题。
有没有办法更改它,以便在 所有 位评论者接受时将评论标记为 "Accept"ed?
even though some reviewers may have open questions
这个有点问题。从根本上说,如果您有一位工程师 忽略 审查时的代码反馈并无论如何推送代码,那么应该单独与该工程师交谈。 Phabricator 中灵活的代码审查的要点是大多数人在大多数时间都可以被信任,并且通过该产品可以提高工作流程速度并保持畅通无阻(和工程师工作)。
如果您不信任您的工程师,像 Herald(添加阻止审阅者)和 Owners(代码所有权包)这样的工具仍然可以实现您可能想要的大部分功能。
https://secure.phabricator.com/T731是上游的任务跟踪"all must accept"类型规则。