ドキュメントには頻繁に更新が加えられ、その都度公開されています。本ページの翻訳はまだ未完成な部分があることをご了承ください。最新の情報については、英語のドキュメンテーションをご参照ください。本ページの翻訳に問題がある場合はこちらまでご連絡ください。

プルリクエストのレビューについて

レビューを使うと、コラボレーターはプルリクエスト中で提案された変更に対してコメントしたり、変更を承認したり、プルリクエストがマージされる前にさらなる変更をリクエストしたりできます。 リポジトリ管理者は、すべてのプルリクエストをマージ前に承認することを必須にできます。

ここには以下の内容があります:

プルリクエストのレビューについて

プルリクエストがオープンされると、読み取りアクセス権を持つ人は誰でもそのプルリクエストが提案する変更をレビューしてコメントできます。また、作者が直接プルリクエストから適用できる変更を、行に対して提案することもできます。詳細は「プルリクエストで提案された変更をレビューする」を参照してください。

リポジトリオーナーとコラボレーターは、特定の人物にプルリクエストのレビューをリクエストできます。 また、Organization メンバーは、リポジトリの読み取りアクセス権を持つ Team にプルリクエストのレビューをリクエストできます。 For more information, see "Requesting a pull request review."

レビューにより、提案された変更についての議論がなされ、その変更がリポジトリのコントリビューションのガイドラインやその他の品質標準を満たすことを保証しやすくなります。 コードの特定の種類や領域に対して、どの個人や Team をオーナーとするかを、CODEOWNERS ファイルで定義できます。 プルリクエストが、定義されたオーナーを持っているコードを変更するものである場合、オーナーである個人あるいはTeam がレビューを担当するよう、自動的にリクエストされます。 詳細は「コードオーナーについて」を参照してください。

凝固メント付きの変更をリクエストするレビューのヘッダ

レビューには 3 つのステータスがあります:

  • Comment: 明示的に変更を承認したり追加の変更をリクエストしたりすることなく、一般的なフィードバックをサブミットします。
  • Approve: フィードバックをサブミットし、プルリクエスト中で提案された変更のマージを承認します。
  • Request changes: プルリクエストをマージする前に対処しなければならないフィードバックをサブミットします。

レビューステータスの画像

ヒント

  • If required reviews are enabled and a collaborator with write, admin, or owner access to the repository submits a review requesting changes, the pull request cannot be merged until the same collaborator submits another review approving the changes in the pull request.
  • リポジトリのオーナーと管理者は、プルリクエストが承認レビューを受けていなかったり、あるいは変更をリクエストしたレビュー担当者がOrganizationを離れていたりアクセスできなくなっている場合でも、プルリクエストをマージできます。
  • 必須レビューと古いレビューの棄却がどちらも有効化されており、承認済みのプルリクエストのブランチにコードを変更するコミットがプッシュされた場合、その承認は却下されます。 そのプルリクエストは、再度レビューされ承認されるまではマージできません。
  • 同じコミットを指す複数のオープンされたプルリクエストがあり、それぞれがheadブランチを持つ場合、いずれかがペンディングあるいは拒否されたレビューを持っているなら、それらはマージできません。

プルリクエストが受けたすべてのレビューは、Conversationタイムラインで見ることができ、リポジトリオーナー及びコラボレーターによるレビューは、プルリクエストのマージボックスで見ることができます。

マージボックス中のレビューの画像

Tip: You can find a pull request where you or a team you're a member of is requested for review with the search qualifier review-requested:[USERNAME] or team-review-requested:[TEAMNAME]. For more information, see "Searching issues and pull requests."

会話を解決する

プルリクエストをオープンしたり、プルリクエストがオープンされたリポジトリへの書き込みアクセス権を持っていたりすれば、プルリクエスト中の会話を解決できます。

Files changed(変更されたファイル)タブ上の会話が完了したことを示すには、Resolve conversation(会話を解決)をクリックしてください。

会話の解決ボタンが付いたプルリクエストの会話

会話全体が畳まれ、解決のマークが付きます。これで、まだ対応が必要な会話を見つけやすくなります。

解決された会話

コメントの示唆がプルリクエストの範囲を超えているなら、そのコメントへのフィードバックやリンクを追跡する新しいIssueをオープンできます。 詳しい情報については、「プルリクエストでのフィードバックを取り込む」を参照してください。

必須のレビュー

Repository administrators can require that all pull requests receive a specific number of approving reviews from people with write or admin permissions in the repository or from a designated code owner before they're merged into a protected branch. 詳しい情報については保護されたブランチについてを参照してください。

必須レビューが有効化されると、リポジトリへのアクセス権を持つ人は誰でもプルリクエスト中の変更を承認できます。 However, you won't be able to merge your pull request until the required number of reviewers with write or admin permissions in the repository approve your pull request's changes in their review. リポジトリの権限レベルに関する詳しい情報については「Organizationのリポジトリの権限レベル」を参照してください。指定されたコードオーナーからのレビューが必須で、プルリクエストが指定されたオーナーのコードに影響するものなら、そのオーナーからの承認が必要です。

詳しい情報についてはプルリクエストのための必須のレビューについてを参照してください。

ヒント: 管理あるいは書き込みアクセスをリポジトリに対して持つ人は、必要に応じてプルリクエストのレビューを却下できます。 詳しい情報についてはプルリクエストレビューの却下を参照してください。

参考リンク

担当者にお尋ねください

探しているものが見つからなかったでしょうか?

弊社にお問い合わせください