17

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

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

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

4

1 に答える 1

20

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

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

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

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

レビュー担当者は、開発者が行ったすべての変更を確認し、それらにコメントして、コメントを開発者に送信できます。すべてが Visual Studio 内で行われ、すべてのコメントは TFS に保持され、後で報告できます。

また、レビューの完了時に、レビュー担当者は、変更に対する意見を「よさそう」、「コメントあり」、または「作業が必要」として指定できます。

これに関する MSDN のこのページを必ずお読みください: http://msdn.microsoft.com/en-us/library/hh474795.aspx

Channel 9 でこのビデオをチェックすることもできます: http://channel9.msdn.com/Series/Visual-Studio-2012-Premium-and-Ultimate-Overview/Visual-Studio-Ultimate-2012-Using-Code-Review -to-Improve-Quality

編集

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

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

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

チェックインを本当に防止する必要がある場合は、開発者のコ​​ミット許可を拒否し、コード レビュー (またはシェルブ セット) を使用して変更をマネージャーに送信する必要があります。その後、マネージャーは変更の保留を解除し、レビュー後にコミットできます。

于 2014-10-28T13:01:09.640 に答える