TFSコードレビュー-コメントへの応答として更新されたファイルを表示する


127

VS 2012およびVS 2013プレビューに組み込まれているコードレビュー機能を使用し始めています。レビューのリクエストとコメントの追加は非常に簡単に思えます。コードの変更をリクエストするコメントを誰かが追加した場合、リクエスターはこれらの変更をどのように行って表示するのですか?

したがって、プロセスは次のように流れます。

  1. Person 1はコードレビューを要求します。
  2. Person 2はコメントを追加し、「Needs Work」を選択します。
  3. 人1が必要な変更を行います。

個人1は、これらの変更を個人2にどのように表示するのですか?コメントを追加して送信できますが、ファイルは変更されません。元のレビューが要求されたときに作成されたチェンジセットからのファイルであると想定しています。Person 1はこのレビューを閉じて、2回目のレビューをリクエストする必要がありますか?

変更を加えた理由についての会話の履歴が失われるため、2番目のレビューを作成することは最適ではないようです。

コードレビューの基本的な機能の使用方法を示すWebサイトはたくさんありますが、ベストプラクティスを示すサイトはありますか?たとえば、ファイルの横にあるチェックボックスをクリックするのは誰ですか。


「ベストプラクティスを示すサイトはありますか」の+1。ALM Rangersは通常、詳細な使用法についての「後藤の人」ですが、私が見つけることができる唯一の情報は、あなたの質問に答えません。Code Reviewを使用するためのウォークスルーとシナリオをいくつか紹介したいと思います。
DaveShaw 2013

回答:


164

したがって、プロセスは次のように流れます。

  1. Person 1はコードレビューを要求します。
  2. Person 2はコメントを追加し、「Needs Work」を選択します。
  3. 人1が必要な変更を行います。
  4. Person 1は、コードレビューに関連付けられたシェルブセットを更新します
  5. ユーザー1がコメントを追加して、ディスカッションを続行します
  6. 承認されるまで手順2〜5を繰り返します。

レビューに関連付けられたシェルブセットを更新するために必要な手順は次のとおりです。

  1. 「Code Review」ペインから「view shelveset」リンクを選択します
  2. [Shelveset Details]ペインで、シェルブセット名を強調表示してコピーします
  3. [保留中の変更]ペインに移動し、[シェルブ]をクリックしてシェルブセット名を貼り付けます
  4. シェルブセットの置換確認ダイアログの[はい]ボタンを押します
  5. これで、レビュー担当者は更新されたファイルを確認でき、レビューディスカッションを続行できます

わかりやすくするために、スクリーンショットをいくつか含めました。


1)「コードレビュー」ペインから、次のように「シェルブセットを表示」リンクを選択します。

ここに画像の説明を入力してください


2)「シェルブセットの詳細」ペインから、次のようにシェルブセット名を強調表示してコピーします。

ここに画像の説明を入力してください


3)[保留中の変更]ペインに移動し、[シェルブ]をクリックして、シェルブセット名を貼り付けます。例:

ここに画像の説明を入力してください


4)シェルブセットの置換確認ダイアログの[はい]ボタンを押します。

ここに画像の説明を入力してください


7
このソリューションの問題は、すべてのコメントが吹き飛ばされるという点で破壊的であるということです。新しいコードレビューを作成し、古いコードレビューを関連アイテムとして設定することが、より良い解決策になると思います。
バルディア

15
シェルブセットを交換しても、コメントだけはファイルに影響しません。私たちのチームはこの方法をうまく使用しており、レビューのコメントを失うことはありません。
チャドディエンハート2014

10
これは機能しますが、ファイルを編集するときに行を追加/削除してシェルブセットを置き換えると、そのファイルにあるコメントは間違った行を指し示します。
Chin

2
私は通常、繰り返したくありませんが、上記の@Chinのポイントにより、この答えは完全に受け入れられなくなります。はい、シェルフセットを簡単に更新できますが、そうすることで、以前のすべてのレビューコメントが誤ったテキストを指すようになります。(どうやらレビューコメントはファイルの先頭からの絶対オフセットに添付されているためです。)
Michael Sorens

7
マニュアルが多すぎて理想的ではありません。TFSまたはVisual Studioには、連続するレビュー間の変更を比較するものが必要です。これにより、レビュー担当者は、最後のレビュー以降に行われた変更を正確に確認できます。このようなものは存在しませんか?たぶんプラグインか何か...
Iúriドスアンジョス

6

正しい手順は、Person 1が変更を行い、別のレビューを要求することです。コードに作業が必要な場合は、コードを変更することになるため、比較のために古いバージョンを確認する必要があります。コメントを確認したい場合は、閉じた後も履歴に古いレビューがあります。現在、職場でのコードレビュープロセスの最適化を進めています。


2

この質問は古いのですが、他のポスターで指摘されているように、まだサポートされていません。chadによって提案されたソリューションは、いくつかの場合に機能しますが、他の場合には奇妙な動作をします。

最近、TFSチームは、非常に古いuservoice要求に対するソリューションの計画段階を開始しました。コードレビューの更新を可能にし、コードレビューに反復を持たせることでこの問題をエレガントに解決します。


1
すごい。私も楽しみにしています。私は本当の解決策を歓迎します。私の投稿は単に問題を回避する方法です。
Chad Dienhart、2015年

うまくいけば、Azure DevOpsのプルリクエストのようには見えません。彼らはスパムの悪夢です。
StingyJack

0

2つの異なるレビューでこれを行う必要があります。ただし、2番目のレビューで履歴を保持する方法もあります。必要なのはタスクだけです。

このワークフローは、チェンジセットベースのレビューで説明されていますが、シェルフベースのレビューでも機能します。

  1. タスク1を作成
  2. 変更セット1をチェックインする前に、関連する作業項目としてtask1を追加します
  3. 関連する1つの作業項目で変更をチェックインし、この変更セットのレビューを要求します
  4. タスク2を作成する
  5. 変更セット2をチェックインする前に、両方のタスクを関連作業項目として追加します。
  6. 関連する2つの作業項目を含む変更をチェックインし、この変更セットのレビューを要求する

これで、2番目のレビューリクエストで、レビュー担当者は関連するタスクを探すことができ、レビュー担当者がtask1を探すと、変更セット1とレビューリクエストがコメントとともに表示されます。したがって、会話の履歴が失われることはありません。

弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.