VSTS中的强制性代码审查

Question

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

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

Answer 1

对于TFVC，您可以使用第三方签到策略，例如适用于您的VS客户端的Colin's ALM Checkin Policies VS 2017（请参阅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。

Answer 2

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

Answer 3

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