VSTS 中的强制代码审查答案

【问题标题】：Mandatory Code Review in VSTSVSTS 中的强制代码审查
【发布时间】：2018-08-13 03:01:03
【问题描述】：

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

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

【问题讨论】：

您是否查看过代码审查支持？ docs.microsoft.com/en-us/vsts/tfvc/get-code-reviewed-vs 我还建议考虑迁移到 Git。我知道这是一大步，但 VSTS 中的 Git 支持非常棒，并且支持拉取请求很容易设置您正在寻找的工作流程。
嗨，谢谢你的建议，是的，在 Git 中我知道我们有一个拉取请求。但组织正在寻求与 TFS 和 VSTS 合作。
您正在将您的项目迁移到 VSTS，并且您的公司希望在此过程中添加一个审核周期。借此机会更改源代码控制！这是正确的时间。在我看来，TFSVC 的其他替代方案太有限/太麻烦了。

【解决方案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 下所需的审阅者

【讨论】：

【解决方案2】：

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

【讨论】：

【解决方案3】：

TFVC 支持代码审查工作流程（尽管它通常被认为远不如 Git 拉取请求/分支策略模型，这是有充分理由的）。然而，没有很好的方法让它成为强制性的。您可以应用第三方签入策略，但需要注意的是签入策略仅在 Visual Studio 中有效，并且相对容易被有决心的个人规避。

【讨论】：