TFSコードレビュー-コメントへの応答として更新されたファイルを表示する
VS 2012およびVS 2013プレビューに組み込まれているコードレビュー機能を使用し始めています。レビューをリクエストしてコメントを追加するのは簡単です。コードの変更を要求するコメントを誰かが追加した場合、リクエスターはどのようにしてこれらの変更を行い、それらを表示しますか？
したがって、プロセスは次のように流れます。
- 人1はコードレビューを要求します。
- 人2はコメントを追加し、「Needs Work」を選択します。
- 人1が必要な変更を行います。
Person 1は、これらの変更をPerson 2にどのように表示しますか？コメントを追加して送信できますが、ファイルは変更されません。ファイルは、元のレビューが要求されたときに作成された変更セットからのものであると想定しています。 Person 1はこのレビューを閉じて、2回目のレビューをリクエストする必要がありますか？
2番目のレビューを作成するのは最適ではないようです。なぜなら、変更を行った理由に関する会話の履歴が失われるためです。
コードレビューの基本機能の使用方法を示すWebサイトは数多くありますが、ベストプラクティスを示すサイトはありますか？たとえば、ファイルの横にあるチェックボックスをクリックする必要があるのは誰ですか？
したがって、プロセスは次のように流れます。
- 人1はコードレビューを要求します。
- 人2はコメントを追加し、「Needs Work」を選択します。
- 人1が必要な変更を行います。
- 担当者1コードレビューに関連付けられているシェルブセットを更新します
- 人1はコメントを追加して議論を続けます
- 承認されるまで手順2〜5を繰り返します
レビューに関連付けられているシェルブセットを更新するために必要な手順は次のとおりです。
- [コードレビュー]ペインから[シェルブセットの表示]リンクを選択します
- [シェルフセットの詳細]ペインで、シェルフセット名を強調表示してコピーします
- [保留中の変更]ペインに移動し、[棚]をクリックして、棚セット名を貼り付けます
- シェルブセット交換確認ダイアログの[はい]ボタンを押します
- これでレビューアは更新されたファイルを見ることができ、レビューの議論を続けることができます
物事を明確にするのに役立つと思うので、いくつかのスクリーンショットを含めました。
1）[コードレビュー]ペインから、次のように[シェルブセットの表示]リンクを選択します。
2）[シェルフセットの詳細]ペインで、次のようにシェルフセット名を強調表示してコピーします。
3）[保留中の変更]ペインに移動し、[棚]をクリックして、次のように棚セット名を貼り付けます。
4）シェルブセット交換確認ダイアログの[はい]ボタンを押します。
正しい手順は、Person 1が変更を行い、別のレビューをリクエストすることです。コードに作業が必要な場合、コードを変更するため、比較のために古いバージョンを使用する必要があります。コメントを確認したい場合は、閉じた後も履歴に古いレビューが残っています。現在、職場でのコードレビュープロセスを最適化しています。
この質問は古いことは知っていますが、他のポスターで指摘されているように、まだサポートされていません。 chadによって提案された解決策は、あるものには機能しますが、他のものには奇妙な動作をします。
最近、TFSチームは、 コードレビューの更新を可能にする非常に古いユーザー音声要求 のソリューションの計画段階を開始しました。
2つの異なるレビューでこれを行う必要があります。ただし、2回目のレビューで履歴を保持する方法もあります。必要なのはタスクだけです。
このワークフローは、チェンジセットベースのレビューについて説明されていますが、シェルフベースのレビューに対しても機能します。
- Task1を作成します
- Changeset1をチェックインする前に、関連する作業項目としてtask1を追加します
- 1つの作業項目に関連する変更をチェックインし、この変更セットにレビューを要求します
- タスク2を作成
- Changeset2をチェックインする前に、両方のタスクを関連する作業項目として追加します。
- 関連する2つの作業項目で変更をチェックインし、この変更セットにレビューを要求します
これで、2番目のレビューリクエストで、レビュアーは関連タスクを検索でき、レビュアーがtask1を検索すると、changeset1とコメント付きのレビューリクエストが表示されます。したがって、会話履歴を失うことはありません。