我如何在tfs 2010中进行代码审查?

时间:2012-06-03 09:09:36

标签: visual-studio-2010 tfs tfs2010

我的Team Foundation Server中有3个级别的用户:

  1. 程序员:代码审核人员必须审核他们的代码,但他们无法签入自己的代码
  2. 专业程序员:他们的代码不需要审核,但他们的代码副本应自动发送给代码审阅者
  3. 代码审核者:他们的代码不需要审核,可以直接签到
  4. 我该如何实现?

4 个答案:

答案 0 :(得分:5)

一种方法是使用分支和警报。例如......

  • 高级程序员可以直接在您的“主”分支上工作,这样他们就可以将代码检查到实时代码库中。
  • 初级程序员可以使用“RequiresReview”分支,然后代码审阅者会审核他们的签到,并且只有当他们对代码满意时才将它们合并到Main。

TFS可以针对各种事件发送电子邮件警报,因此代码审阅者可以订阅签到事件,以了解代码何时已经检入Main(由高级程序员)或需要进行审核(已经过检查)需要由初级程序员审核)

以这种方式使用分支意味着你必须在两个方向合并(将大三的签到合并到Main,并将高级签到合并回到RequiresReview)

初级程序员的另一种方法是使用ShelveSets而不是另一个分支。在这种方法中,Juniors不会检查他们的代码,而只是将其移动到shelveset,然后通过电子邮件发送给代码审阅者,要求他们检查并检查他们的shelveset。这样就不需要将Main中的代码合并到RequiresReview中 - 后者将在主分支上“工作”,但是他们的签到将通过shelveset“门控”。

您还可以购买一些第三方工具,以更加集成的方式为您管理代码审核流程。网络搜索应该可以帮助您找到这些工具。

修改 请注意,Visual Studio 2012和2013现在包含对代码评审的集成支持(基本上是上面搁置设置理念上的正确用户界面),因此从2010年升级可能是比我上面建议的解决方案更好的选择。

答案 1 :(得分:3)

您可以在TFS 2010中对此进行配置,但没有简单的方法可以从UI公开此内容。 TFS 2012和VS2012将为代码审查提供更好的用户界面,它有一个嵌入式流程,可以帮助您完成整个过程。

如果您想如上所述配置源代码管理权限,则可以使用以下方法:

  

编程

     

将它们粘贴到TFS组或AD组并设置其安全性,以便它们在其工作分支上具有Checkout和Lock权限,但没有签入权限。当他们完成他们的工作后,他们会创建一个shelveset,然后让他们以shelveset名称向审阅者发送电子邮件。审阅者要么返回带有注释的集合(内联或通过电子邮件或工作项),要么代表程序员检查代码。因为程序员不直接签到,所以无法设置自动警报。您可以更改流程模板,使其可以作为任务的状态进行审核,并让他们转换任务(这是自定义代码审核解决方案的工作方式)。

     

Code Reviewer

     

必须至少拥有结帐,锁定和签入权限。将它们粘贴在TFS组或AD组中并相应地设置安全性。他们可以直接办理登机手续,无需使用货架。

     

专业程序员

     

(我会小心这个词,它会让其他程序员听起来不专业,我更喜欢高级或经验丰富)。授予他们与Code Reviewer相同的权限。 Configure a TFS Alert当该组中的任何人检查某些内容时会发出警告。将警报发送给代码审阅者。然后,他们可以使用“查看历史记录”选项查看“更改集”并将评论发送给相关程序员。使用TFS Power Tools可以更轻松地配置警报(并且功能更强大)。

如果您想要一个更高级的解决方案,一个集成到TFS流程模板中的解决方案,那么您可以使用这两个开源解决方案中的一个:

或者看看他们的代码(他们是开源代码)来了解如何自己构建这样的东西。但这并不是一件容易的事情,一旦TFS 2012问世,它可能会毫无用处。

同时查看my presentation on the new features in TFS 2012 around code review(ppt是英文版,但录制内容是在荷兰会议上进行的,因此是荷兰语。)

答案 2 :(得分:2)

+1杰森 好的第三方之一是团队审查

http://teamreview.codeplex.com/

答案 3 :(得分:1)

如果您可以升级到Visual Studio 2012,那么现在可以在Team Explorer UI中内置一个很棒的代码审查界面(也需要TFS 2012)。看看:

http://blogs.msdn.com/b/mvpawardprogram/archive/2012/09/04/new-code-review-feature-in-visual-studio-2012.aspx