visual-studio-2013 - checkin TFS 2013 之前的代码审查

Question

我正在尝试实现一个流程，以便经理可以查看
所有开发人员的代码，然后开发人员才能 checkin TFS 2013。

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

我将 TFS 2013 与 Visual Studio 2013 一起使用。

Answer 1

TFS 2013 开箱即用地支持这一点，并且使用起来非常简单。

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

转到团队资源管理器

未决更改

在“操作”菜单下，选择请求审核

指定审阅者，然后发送。

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

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

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

请确保您在 MSDN 上阅读了此页面:
http://msdn.microsoft.com/en-us/library/hh474795.aspx

您还可以在第 9 channel 观看此视频:
http://channel9.msdn.com/Series/Visual-Studio-2012-Premium-and-Ultimate-Overview/Visual-Studio-Ultimate-2012-Using-Code-Review-to-Improve-Quality

编辑

为了防止未经审查就 checkin ，您可以使用 checkin 政策。这是一个自定义 checkin 政策，需要在 checkin 前进行代码审查:

https://visualstudiogallery.msdn.microsoft.com/c476b708-77a8-4065-b9d0-919ab688f078

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

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