签入 TFS 2013 之前的代码审查答案

【问题标题】：Code Review before checking in to TFS 2013签入 TFS 2013 之前的代码审查
【发布时间】：2016-10-27 01:06:02
【问题描述】：

我正在尝试实施一个流程，以便经理可以审核在开发人员签入 TFS 2013 之前，所有开发人员的代码。

在将代码添加到实际项目之前，是否有任何流程需要人工审核？

我将 TFS 2013 与 Visual Studio 2013 结合使用。

【问题讨论】：

【解决方案1】：

TFS 2013 开箱即用支持此功能，使用起来非常简单。

开发者需要在团队资源管理器中提交审核请求：

目标用户（在您的情况下为经理）将收到通知（如果您已在 TFS 上配置了电子邮件设置），并且还可以在团队资源管理器 > 我的工作中查看审核请求。

Reviewer 可以看到开发人员所做的所有更改，对其进行评论，并将 cmets 发送给开发人员。整个事情发生在 Visual Studio 内部，所有的 cmets 都保存在 TFS 上，以后可以报告。

此外，在完成审核后，审核者可以将他/她对更改的意见指定为“看起来不错”、“有评论”或“需要工作”。

编辑

要防止未经审核就签入，您可以使用签入政策。这是一个自定义签入政策，需要在签入前进行代码审查：

但我应该指出，签入策略不是安全机制，请将其视为开发人员发送签入代码的提醒。开发人员可以轻松覆盖它们。

如果您确实需要防止任何签入，您应该拒绝开发人员的提交权限，并让他们使用代码审查（或搁置集）将更改发送给经理。然后，经理可以在审核后取消搁置更改并提交。

【讨论】：

是的，你是对的。但是 Visual Studio 显示了“请求审查”和“签入”两个选项，开发人员可以在不审查的情况下签入代码。我想要的是开发人员要求审核，经理在审核后签入代码。并且开发人员无法在不要求审查的情况下签入代码。
需要注意的是，目前代码审查只能由 VS Premium 和 Ultimate 用户请求和查看，VS Professional 不能。
@Jazaret 请注意，从 VS2015 开始，社区版、专业版和企业版的用户都可以使用代码审查工具
关于最后一个建议，关于搁置的建议... 这将导致所有更改都被记录为已由经理完成。是否有一种简单、直观的方式可以代表其他用户进行签到？我看到有一个权限设置，但是除了使用模拟或命令行之外，我找不到任何关于如何做到这一点的文档。
@Iravanchi 正如您在此处提到的，“操作”菜单中没有“请求审核”选项。