VSTS中的强制性代码审查

Posted 2021-05-05

我们正在VSTS中迁移我们的项目，我们的团队中有一个层次结构。是否有可能开发人员只有在团队领导审核代码后才能签入代码。在开发人员签到之前，会通过变更集的详细信息向团队负责人触发通知/电子邮件，并且只有在批准/审核评论后他才能办理登机手续。

我正在使用Visual Studio 2017，TFS和VSTS。

答案

对于TFVC，您可以使用第三方签到策略，例如Colin's ALM Checkin Policies VS 2017，它适用于您的VS客户端（请参阅How to utilize Team Foundation Server check-in policies）。但它是客户端策略，这意味着您需要将策略应用于每个客户端。

因此，要强制执行/强制执行代码审查，您需要编写服务器端策略。只需参考这篇文章：How to Enforce Check-In Policies和TFS Server-side check-in policy for git repositories编写自己的签入政策。

对于Git，通过设置分支策略来强制代码审查会更容易：

启用Require a minimum number of reviewers并在Require a minimum number of reviewers下添加团队负责人作为必需的审阅者

有关详细信息，请参阅Improve code quality with branch policies。

另一答案

TFVC支持Code Review工作流程（虽然它通常被认为远远低于Git pull请求/分支策略模型，但有充分的理由）。然而，没有好办法让它成为强制性的。您可以应用第三方签到策略，但需要注意的是，签入策略仅在Visual Studio中有效，并且相对容易被确定的个人规避。

另一答案

尝试使用Colin's ALM Checkin Policies VS 2017进行团队基础版本控制。