プルリクエストのレビューについて
pull request を開いた後、読み取り アクセス権を持つすべてのユーザーは、提案された変更をレビューしてコメントできます。 また、作者がプルリクエストから直接適用できるコード行への特定の変更を提案することもできます。 詳細については、「pull request で提案された変更をレビューする」を参照してく� さい。
リポジトリオーナーとコラボレーターは、特定の人物にプルリクエストのレビューをリクエストできます。 また、Organization メンバーは、リポジトリの読み取りアクセス権を持つ Team にプルリクエストのレビューをリクエストできます。 詳細については、「Pull Request レビューをリクエストする」を参照してく� さい。 チー� メンバーのサブセットを指定して、チー� 全体に代わって自動的に割り当てることができます。 詳細については、「チー� のコード レビュー設定の管理」を参照してく� さい。
レビューにより、提案された変更についての議論がなされ、その変更がリポジトリのコントリビューションのガイドラインやその他の品質標準を満たすことを保証しやすくなります。 コードの特定の種類や� �域に対して、どの個人や Team をオーナーとするかを、CODEOWNERS ファイルで定義できます。 プルリクエストが、定義されたオーナーを持っているコードを変更するものである� �合、オーナーである個人あるいはTeam がレビューを担当するよう、自動的にリクエストされます。 詳細については、「コードオーナーについて」を参照してく� さい。
レビューには 3 つのステータスがあります:
- コメント: 変更を明示的に承認したり、追� の変更を要求したりせずに、一般的なフィードバックを送信します。
- 承認: フィードバックを送信し、pull request で提案された変更のマージを承認します。
- 変更の要求: pull request をマージする前に対処する必要があるフィードバックを送信します。
ヒント:
- 必要なレビューが有効になっており、リポジトリへの 書き込み、 管理者、または 所有者 アクセス権を持つコラボレーターが変更を要求するレビューを送信した� �合、同じコラボレーターが pull request の変更を承認する別のレビューを送信するまで、pull request をマージすることはできません。
- リポジトリのオーナーと管理者は、プルリクエストが承認レビューを受けていなかったり、あるいは変更をリクエストしたレビュー担当者がOrganizationを離れていたりアクセスできなくなっている� �合でも、プルリクエストをマージできます。
- 必� �レビューと古いレビューの棄却がどちらも有効化されており、承認済みのプルリクエストのブランチにコードを変更するコミットがプッシュされた� �合、その承認は却下されます。 そのプルリクエストは、再度レビューされ承認されるまではマージできません。
- 同じコミットを指す複数のオープンされたプルリクエストがあり、それぞれがheadブランチを持つ� �合、いずれかがペンディングあるいは拒否されたレビューを持っているなら、それらはマージできません。
- 書き込みまたは管理者のアクセス許可を持つ個人からのレビューの承認を必� �としているリポジトリの� �合、これらのアクセス許可を持つ個人からの承認には緑色のチェック マークが表示され、これらのアクセス許可を持たない個人からの承認には灰色のチェック マークが表示されます。 灰色のチェック マークが付いた承認は、pull request をマージできるかどうかに影響しません。
- Pull Requestの作者は、自分自身のPull Requestを承認することはできません。
プルリクエストが受けたすべてのレビューは、Conversationタイ� ラインで見ることができ、リポジトリオーナー及びコラボレーターによるレビューは、プルリクエストのマージボックスで見ることができます。
ヒント: 検索修飾子 review-requested:[USERNAME]
または team-review-requested:[TEAMNAME]
が使用された、自分または自分がメンバーであるチー� が確認を求められているプルリクエストを探すことができます。 詳細については、「Searching issues and pull requests」 (問題とプルリクエストの検索) を参照してく� さい。
会話を解決する
プルリクエストをオープンしたり、プルリクエストがオープンされたリポジトリへの書き込みアクセス権を持っていたりすれば、プルリクエスト中の会話を解決できます。
[変更されたファイル] タブの会話が完了したことを示すには、 [会話の解決] をクリックします。
会話全体が畳まれ、解決のマークが付きます。これで、ま� 対応が必要な会話を見つけやすくなります。
コメントの示唆がプルリクエストの範囲を超えているなら、そのコメントへのフィードバックやリンクを追跡する新しいIssueをオープンできます。 詳細については、「コメントから Issue を開く」を参照してく� さい。
会話の発見とアクセス
[変更されたファイル] タブの上部に表示される [会話] メニューを使用すると、pull request 内のすべての会話を検出して移動できます。
このビューから、未解決の会話、解決済みの会話、古くなった会話を見分けることができます。 これによって、会話を見つけて解決することが容易になります。
レビューを再リクエストする
たとえばPull Requestに相当の変更を� えた後に、レビューを再度リクエストできます。 レビュー担当者からの新しいレビューをリクエストするには、 [会話] タブのサイドバーで、 アイコンをクリックします。
必� �のレビュー
リポジトリ管理者はすべてのPull Requestに対し、保護されたブランチにPull Requestを誰かがマージできるようになる前に受けなければならない承認レビューの数を指定できます。 リポジトリに書き込み権限を持っている人か、指定されたコードオーナーからの承認レビューを必� �とすることができます。詳細については、「保護されたブランチについて」を参照してく� さい。
ヒント: 必要に応じて、リポジトリへの 管理者 または 書き込み アクセス権を持つユーザーは、pull request レビューを無視できます。 詳細については、「プル リクエスト レビューの却下」を参照してく� さい。
参考資料
- 「Reviewing proposed changes in a pull request」 (pull request で提案された変更をレビューする)
- 「pull request レビューを表示する」
- 「リポジトリ共同作成者のガイドラインの設定」