2013-10-09 56 views
7

我正在尝试为我的团队的新TFS 2012服务器配置代码审查过程。目前有些事情我们无法弄清楚如何在TFS/VS 2012中使用综合代码审查功能:TFS 2012的代码审查替代方案?

  1. 要求对任何签入的代码进行审批/最终代码审查。更具体地说,只要他们在没有相关代码审查的情况下签入代码,就会提醒开发人员,直到他们解决它。
  2. 允许开发人员请求代码审查,允许审阅者发表评论并将审阅标记为“需要工作”,然后允许开发人员更改代码并将其与原始代码审阅相关联,发表评论,然后将其发回给审阅者,并且无限期地重复此过程直到审阅者批准代码审阅。我们不希望进行新的代码审查,并将它们与TFS的工作流程建议的代码审查相关联。

据我所知,TFS 2012综合代码审查过程的破产者是子弹#2,我找不到解决这个问题的任何地方,听起来好像没有办法绕过它。

所以我在寻找的是一种替代的第三方代码审查解决方案,它可以集成到我们的TFS源代码控制中,或者可以改变TFS以满足我们的需求。

+0

对于#2我已经发布了一种方法来执行此操作:http://stackoverflow.com/questions/18322791/tfs-code-reviews-show - 更新文件回应评论/ 21126092#21126092 不幸的是,它需要审查请求者一些额外的步骤。 –

回答

4

你是对的,这是人们在尝试使用TFS代码审查功能时遇到的两个常见缺陷。

#1,执行代码审查政策,它不支持开箱即用,但科林Dembovsky创造了一个自定义签入策略,可以强制执行此:http://visualstudiogallery.msdn.microsoft.com/c476b708-77a8-4065-b9d0-919ab688f078

#2,TFS目前不支持“多次迭代”代码评论。今天在TFS上处理这个问题的唯一方法就是启动一个新的Code Review。

Dev Requests Code Review 
Reviewer provides comments 
Dev Closes Code Review 
Dev fixes code 
Dev Creates new Code Review 
... 
+2

谢谢您提供的信息,不幸的是#2对我们来说是一个破坏者。 –

3

我可以建议Review Assistant作为替代TFS代码审查功能。

它完全实现了你的#2点 - “多次迭代”代码评论。

点#1(自定义签入策略)尚未实施,但它在我们的路线图上。

Review Assistant如果集成到Visual Studio以及标准功能中。但是你可以将它与专业版的VS一起使用,而不仅限于Premium或Ultimate。

+0

我们安装并评估了Review Assistant,但不幸的是它不支持签入后代码审查政策,我们需要能够强制每次签入都包含在代码审查中,否则应该提醒变更集的作者并且我们需要能够查看任何未发现代码的总体报告。另一个问题是我们通常在代码评论中包含单元测试,并且无法将自定义文本字段添加到代码评论中以支持此操作。 –

+0

正如我所提到的,目前我们不支持审查政策,但我们计划。但是单元测试的问题并不清楚。我们希望进一步解释。 – Devart

+0

我们要求任何代码更改都包含在代码审查中,并且每个代码审查都与单元测试相关联。在我们的例子中,能够将代码审查链接到TFS中的单元测试/测试用例工作项目就足够了。 –

0

在TFS中,先前的代码评论将作为关联的工作项添加到当前的代码评审中。我鼓励使用签入过程来关闭列为“看起来不错”的最后一个代码评论。这将代码检查关闭代码审查。