レビューの解決

pull request を開くと、レビュー担当者は、マージ前にコードの改善に役立つフィードバックを残します。そのフィードバックを解決するには、各コメントを理解し、必要な変更を行い、すべてに対処していることを確認します。

レビューフィードバックについて

校閲者は、一般的なフィードバックを残し、特定の行にコメントし、提案として正確な変更を提案することができます。レビューは pull request タイムラインに表示されるため、ディスカッションに従って、まだ応答が必要なコメントを確認できます。リポジトリの設定に応じて、レビュー担当者は、マージする前に対処する必要がある pull request または要求の変更を承認することもできます。

何かを変更する前に、各コメントを読んで意図を理解してください。コメントは、バグを指摘したり、質問したり、別のアプローチを要求したり、直接受け入れることができる編集を提案したりする場合があります。

Copilotにアクセスできる場合は、レビューコメントを解釈し、修正プログラムを提案するのに役立ちます。これは、pull request に多くのコメントが含まれている場合に役立ちます。

変更の実装とコードの修正

フィードバックに対処すると、次のことができます。

レビュアーが提案した変更を直接適用すると、その提案内容があなたのブランチにコミットされます。
より広範な編集をローカルで行い、新しいコミットをブランチにプッシュします。

ほとんどのフィードバックは、コードを更新し、新しいコミットを pull request ブランチにプッシュすることで解決されます。フィードバックに対処するときは、会話を解決済みとしてマークして、自分と校閲者の両方が、何が行われ、何がまだ注意が必要かを追跡できるようにします。

必須レビュアーが承認し、要求された変更が残っていない場合、プルリクエストはマージに進む準備が整っています。 pull request はブランチを追跡するため、新しいコミットのたびに pull request が更新され、自動チェックが再実行されます。

フィードバックを効率的に処理する

より大きい、またはより詳細なプル要求では、いくつかのプラクティスがフィードバックに迅速に対処するのに役立ちます。

提案された変更をバッチに追加して提案を受け入れるバッチ。そのため、複数の受け入れられた変更が、提案ごとに 1 つのコミットではなく、1 つのコミットに反映されます。
マージ競合は事前に解決しておきましょう。そうすれば、プルリクエストのレビューが完了したらすぐにマージできます。
大幅な変更を加えた後はレビューを再依頼して、レビュー担当者にプルリクエストをもう一度確認できる状態になったことを知らせます。
必要なレビューを理解する。 ブランチで承認またはコードオーナーのサインオフが必須の場合、「変更をリクエスト」レビューまたは却下された承認レビューは、解決されるまでマージをブロックすることがあります。
pull request を展開するのではなく、コメントにリンクする問題を開いて、スコープ外のフィードバックを追跡します。