17

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

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

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

4

1 回答 1

20

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

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

  • 转到团队资源管理器
  • 打开待处理的更改
  • 在“操作”菜单下，选择请求审核
  • 指定审阅者，然后发送。

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

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

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

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

您还可以在第 9 频道观看此视频：http: //channel9.msdn.com/Series/Visual-Studio-2012-Premium-and-Ultimate-Overview/Visual-Studio-Ultimate-2012-Using-Code-Review提高质量

编辑

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

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

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

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

于 2014-10-28T13:01:09.640 回答