web-dev-qa-db-ja.com

TFS 2013にチェックインする前のコードレビュー

開発者がTFS 2013にチェックインする前に、マネージャーがすべての開発者のコ​​ードをレビューできるように、プロセスを実装しようとしています。

実際のプロジェクトに追加する前に、人間によるコードのレビューを要求するプロセスはありますか?

Visual Studio 2013でTFS 2013を使用しています。

15
Amaan Khan

TFS 2013はこれをすぐにサポートしており、非常に簡単に使用できます。

開発者は、チームエクスプローラーでレビューのリクエストを提出する必要があります。

  • チームエクスプローラーに移動
  • 保留中の変更を開く
  • [アクション]メニューで、[レビューをリクエスト]を選択します
  • レビュー担当者を指定して送信します。

ターゲットユーザー(この場合はマネージャー)に通知され(TFSで電子メール設定を構成している場合)、チームエクスプローラー>マイワークでレビューリクエストを確認することもできます。

レビュー担当者は、開発者によって行われたすべての変更を確認し、それらにコメントし、コメントを開発者に送信できます。すべてが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 -コードレビューから改善品質


編集

レビューなしのチェックインを防ぐには、チェックインポリシーを使用できます。以下は、チェックイン前にコードのレビューが必要なカスタムチェックインポリシーです。

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

しかし、チェックインポリシーはセキュリティメカニズムではないことを指摘する必要があります。開発者がチェックイン用のコードを送信することを思い出させるものとして考えてください。開発者は簡単にオーバーライドできます。

チェックインを本当にpreventする必要がある場合は、開発者のコ​​ミット許可を拒否し、コードを使用して変更をマネージャーに送信するようにしてください。レビュー(または棚にセット)。マネージャーは、変更を保留解除し、確認後にコミットできます。

18
Iravanchi