機能ブランチの使用を開始しており、関連するコード レビューがある場合にのみベースラインへのチェックインを許可するチェックイン ポリシーを設定したいと考えています。
2012 年の新しいコード レビュー ワークフローは非常に優れています。開発者や他のレビュー担当者と簡単にやり取りしたり、コード行に直接コメントを付けたりできるからです。それにもかかわらず、次の問題に簡単に遭遇するため、MS はユースケースを十分に考えていないようです。
開発者は、機能ブランチのチェックイン/シェルビングと前方統合に定期的に取り組んでいます。
機能を統合したい場合、彼女はベースラインにマージし直し、これらの保留中の変更のレビューを要求します。
レビュー担当者はいくつかのコメントを作成し、コードを変更する必要があります。彼女はどこでこれをしますか？
オプション 1:ブランチに戻り、コードを編集して、ブランチの変更をチェックインします。最初のマージの保留中の変更を元に戻します。マージして、再度レビューをリクエストしてください。コメントがなくなるまで繰り返します。マージをチェックインします。すべてのレビュー コメントがマージの保留中の変更にあり、彼女はコメントを直接見ないブランチで作業する必要があるため、これはあまり良くありません。
オプション 2:マージの保留中の変更を直接編集します。再審査をリクエストします。コメントがなくなるまで繰り返します。マージをチェックインします。彼女がブランチで作業を続けたい場合は、レビューからの変更がそこにないため、前方統合を行う必要があります。
いずれにせよ、2 回目のレビューは常に非常に煩わしいものです。なぜなら、常にベースラインと比較しているため、1 回目と 2 回目のレビューの間の変更のみを確認する方法がないからです。
ここで何か不足していますか？レビューからの変更をレビューできる別のオプションはありますか? 機能の分岐とコードのレビューのより良い方法を持っている人はいますか?
新規: VS と TFS2013 を使用しても、まだ改善されていません :(