TFSのコードレビューからコードをチェック

Question

コードレビューが承認された後、コードをチェックインしたいと思います。私はこのstackを読んでコードレビューを作成してチェックインしていましたが、私の質問は少し異なります。

私の問題は、コードレビューを作成することです。しかし、私は承認されるまでコードをチェックインしたくありません。そのため、関連する作業項目を削除して別のコードレビューを開始することができなくなります。私は何をしたいのは、コードレビューを作成し、

チームエクスプローラ

にコードレビュー]タブから確認している可能ということですか？ creating a code review after the check inと同じ原則ですが、コードレビューとチェックインがあります。関連するアイテムを削除している可能性があるため、変更が保留されていてチェックインしたくないです。しかし、私は自分のコードレビューに結びついてチェックインしたい。

Answer 1

さて、文書化のためです。私は、TFSが私に許す棚を完全に理解していませんでした。 Shelve and Unshelve Pending Changesを読んだら、それは私にはもっと意味があります。私は私が作業しているものを棚上げすることができ、コードレビューを行ったコードをunshelveし、そのコードをチェックインします。そうすれば、コードレビューを作成し、そのコードレビューが承認されるまで作業を続けることができます。承認されたら、変更を取り消してチェックインすることができます。

Answer 2

残念ながら、あなたがしようとしていることを行うための「適切な」方法はありません。作業ディレクトリを共有ドライブに置いて、レビュープロセスを開始する準備ができたらレビュー担当者に通知するだけですが、TFS内に開発/レビューの各反復が正式に記録されないようにすることで説明責任を補うことができます。これは、あなたの仕事をチェックインして、査読者に仕事をさせ、査読者が要求した変更を加えてチェックインし、別のコードレビューを得るようにしなければならないことを意味します。

私はここでも私のコメントから私の提案を完全にしたいと思います。

あなたの開発を行い、コードをレビューするために、自己完結型の短命の開発ブランチを作成することをお勧めします。その後、開発とレビューが完了すると、そのブランチを統合して破棄することができます。これにより、よりクリーンで安全なアプローチが提供されます。 1）TFS内の履歴の混乱を軽減します。 2）複数の不要な自動化されたビルド/テスト/などがトリガーされることを防ぎます。

あなたのコメントでは、これが "分岐方法の構造"を変更することをお勧めします。私はこれをどうやって変えるのかは分からない。あなたのマージは、この時点までにすべてのレビューが完了し、1回のきれいなチェックインを実行していることを除いて、最終的な開発チェックインのようになります。これにはチェックイン情報とレビュー情報がすべて含まれていますが、複雑なチェックインチェーンの代わりに、その特定のタスクで実行されたすべてのものを含む単一の折りたたみノードがあります。

あなたのマネージャー、コードレビュー担当者、TFSの担当者、TFSポリシーの作成/保守担当者に確認します。このアプローチは、残りのプロセスがどのように機能するかに関して、実際には何も変わりません。開発サイクルを自己完結型の環境に抽象化しただけです。 2番目のマージを実行すると、今すぐ通常のプロセスに戻ります。