2013-10-09 17 views
7

チームの新しいTFS 2012サーバーのコードレビュープロセスを構成しようとしています。現在、TFS/VS 2012で統合コードレビュー機能を使用する方法を理解できないことがいくつかあります。コードレビューTFS 2012の代替案?

  1. チェックインされたコードの承認済みコードレビューが必要です。具体的には、開発者が解決するまで、関連するコードレビューなしでコードをチェックインしたときはいつでも、開発者に警告します。
  2. 開発者がコードレビューをリクエストできるようにし、レビュー担当者がコメントを作成してレビューを「必要がある」とマークしてから、コード変更を行い、元のコードレビューと関連付けてコメントを付け、レビュー担当者がコードレビューを承認するまで、このプロセスを無期限に繰り返すようにしてください。 TFSのワークフローが示唆するように、新しいコードレビューを作成して以前のコードレビューにリンクすることは望ましくありません。

TFS 2012の統合されたコードレビュープロセスのためのdealbreakerは、箇条書き#2ですが、私はこの問題の解決策はどこにもありません。

私が探しているのは、私たちのTFSソースコントロールに統合できる代替のサードパーティコードレビューソリューションか、TFSを変更して要件を満たす方法です。

+0

#2については、これを行う方法を投稿しました:http://stackoverflow.com/questions/18322791/tfs-code-reviews-show更新されたファイルのコメント/ 21126092#21126092 残念ながら、審査依頼者はもう少し手間を要します。 –

答えて

4

TFSコードレビュー機能を試してみると、人が遭遇する2つの共通の欠点です。 #1のために

、コードレビューポリシーを適用、それは箱から出してサポートされていないが、コリン・Dembovskyは、カスタムチェックインポリシーこの強制することができます作成​​しました:#2についてhttp://visualstudiogallery.msdn.microsoft.com/c476b708-77a8-4065-b9d0-919ab688f078

を、TFSは現在ありません「複数反復」のコードレビューをサポートします。今日のTFSでこれを処理する唯一の方法は、新しいコードレビューを開始することです。

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

情報をありがとうございます、残念ながら#2は私たちのためのdealbreakerです。 –

3

TFSコードレビュー機能の代替として、Review Assistantを提案できます。

#2ポイントの「複数反復」コードレビューを完全に実装しています。

ポイント1(カスタムチェックインポリシー)はまだ実装されていませんが、ロードマップ上にあります。

標準アシスタントと同様に、Visual Studioに統合されている場合は、アシスタントを確認します。しかし、Premium EditionやUltimateだけでなく、Professional EditionのVSでも使用できます。

+0

レビューアシスタントをインストールして評価しましたが残念なことにポストチェックインコードレビューポリシーはサポートされていないため、すべてのチェックインがコードレビューでカバーされるようにする必要があります。そうでなければ、チェンジセットの作成者に警告し、検出されなかったコードの全体的なレポートを見ることができます。もう1つの問題は、通常、コードレビューで単体テストが含まれており、これをサポートするコードレビューにカスタムテキストフィールドを追加する方法がないことです。 –

+0

私が言及したように、現在のところ、審査方針はサポートしていませんが、計画しています。しかし、単体テストの問題は明確ではありません。私たちはいくつかのさらなる説明に感謝します。 – Devart

+0

コードの変更はコードレビューの対象となり、すべてのコードレビューはユニットテストと関連している必要があります。私たちの場合、コードレビューをTFSのユニットテスト/テストケースワークアイテムにリンクすることで十分です。 –

0

TFSでは、以前のコードレビューが現在のコードレビューに関連する作業項目として追加されます。チェックインプロセスを使用して、「よく見える」と表示されている最後のコードレビューを終了することをお勧めします。これは、コードレビューの終了をコードチェックインとしてマークします。