プルリクエストのレビューについて
レビューを使うと、コラボレーターはプルリクエスト中で提案された変更に対してコメントしたり、変更を承認したり、プルリクエストがマージされる前にさらなる変更をリクエストしたりできます。 リポジトリ管理者は、すべてのプルリクエストをマージ前に承認することを必須にできます。
ここには以下の内容があります:
プルリクエストのレビューについて
プルリクエストがオープンされると、読み取りアクセス権を持つ人は誰でもそのプルリクエストが提案する変更をレビューしてコメントできます。詳細は「プルリクエストで提案された変更をレビューする」を参照してください。
リポジトリオーナーとコラボレーターは、特定の人物にプルリクエストのレビューをリクエストできます。 また、Organization メンバーは、リポジトリの読み取りアクセス権を持つ Team にプルリクエストのレビューをリクエストできます。 詳細は「プルリクエストのレビューをリクエストする」を参照してください。
レビューにより、提案された変更についての議論がなされ、その変更がリポジトリのコントリビューションのガイドラインやその他の品質標準を満たすことを保証しやすくなります。 コードの特定の種類や領域に対して、どの個人や Team をオーナーとするかを、CODEOWNERS ファイルで定義できます。 プルリクエストが、定義されたオーナーを持っているコードを変更するものである場合、オーナーである個人あるいはTeam がレビューを担当するよう、自動的にリクエストされます。 詳細は「コードオーナーについて」を参照してください。
レビューには 3 つのステータスがあります:
- Comment: 明示的に変更を承認したり追加の変更をリクエストしたりすることなく、一般的なフィードバックをサブミットします。
- Approve: フィードバックをサブミットし、プルリクエスト中で提案された変更のマージを承認します。
- Request changes: プルリクエストをマージする前に対処しなければならないフィードバックをサブミットします。
参考:
- 必須レビューが有効化されており、リポジトリへの書き込み、管理、オーナー 権限を持つコラボレーターがレビューを要求する変更をサブミットしたなら、そのプルリクエストは同じコラボレーターがプルリクエスト中の変更を承認する他のレビューをサブミットするまではマージできません。
- リポジトリのオーナー及び管理者は、プルリクエストが承認レビューを受け取っていなくても、あるいは変更を要求したレビューアが Organization を離れたりいなくなっていたりしても、プルリクエストをマージできます。
- 必須レビューと古いレビューの却下がどちらも有効化されており、コードを変更するコミットが承認されたプルリクエストのブランチにプッシュされたなら、その承認は却下されます。そのプルリクエストをマージするには、再度レビューと承認が行われなければなりません。
- 複数のオープンなプルリクエストが、それぞれ同じコミットを指す head ブランチを持っているなら、いずれかがペンディングあるいは拒否されたレビューを持っている場合、それらをマージすることはできません。
プルリクエストが受けたすべてのレビューは、Conversationタイムラインで見ることができ、リポジトリオーナー及びコラボレーターによるレビューは、プルリクエストのマージボックスで見ることができます。
参考: あなた、もしくはあなたがメンバーになっている Team がレビューを求められているプルリクエストは、review-requested:[USERNAME]
あるいは team-review-requested:[TEAMNAME]
という検索条件で検索できます。詳細は「Issue およびプルリクエストを検索する」を参照してください。
必須のレビュー
リポジトリの管理者は、プルリクエストが保護されたブランチにマージされる前に、書き込みあるいは管理権限をそのリポジトリに持っている人々か、指定されたコードオーナーから指定された数の承認レビューを受けることを要求できます。詳細は「保護されたブランチについて」を参照してください。
必須レビューが有効化されると、リポジトリへの書き込み権限を持っている人なら誰でも、プルリクエスト中の変更を承認できます。ただし、そのリポジトリの書き込みあるいは管理権限を持っている必要な数のレビューアがレビューでプルリクエストの変更を承認しないかぎり、プルリクエストはマージできません。リポジトリの権限レベルに関する詳しい情報については「Organizationのリポジトリ権限レベル」を参照してください。指定されたコードオーナーからのレビューが必要で、プルリクエストが指定されたオーナーを持つコードに影響するなら、そのオーナーからの承認が必須です。
詳しい情報についてはプルリクエストのための必須のレビューについてを参照してください。
Tip:必要であれば、管理あるいは書き込みアクセスをリポジトリに対して持つ人は、プルリクエストのレビューを却下できます。 詳しい情報についてはプルリクエストレビューの却下を参照してください。