2013-10-09 54 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中,先前的代碼評論將作爲關聯的工作項添加到當前的代碼評審中。我鼓勵使用簽入過程來關閉列爲「看起來不錯」的最後一個代碼評論。這將代碼檢查關閉代碼審查。