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

プルリクエストで提案された変更をレビューする

Pull Request では、コミット、変更されたファイル、ベース ブランチと比較ブランチでのファイル間の違い (つまり "diff") をレビューしたり議論したりできます。

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

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

プルリクエストの変更は、1 ファイルごとにレビューできます。 プルリクエストでファイルを確認しているときに、特定の変更について個別のコメントを残すことができます。 各ファイルの確認が終了したら、ファイルを閲覧済みとしてマークできます。 これによりファイルが折りたたまれるので、まだレビューを必要とするファイルを特定するのに役立ちます。 プルリクエストヘッダのプログレスバーには、閲覧したファイル数が表示されます。 必要な数のファイルを確認した後、要約コメントを付けて確認を送信することにより、プルリクエストを承認するか、追加の変更をリクエストできます。

Tip: あなた、もしくはあなたがメンバーになっているTeamがレビューを求められているプルリクエストは、review-requested:[USERNAME]あるいはteam-review-requested:[TEAMNAME]という検索条件で検索できます。 詳しい情報については「Issueやプルリクエストの検索」を参照してください。

レビューを開始する

  1. リポジトリ名の下にある Pull requests(プルリクエスト)をクリックしてください。 Issue とプルリクエストのタブの選択
  2. プルリクエストのリストで、レビューしたいプルリクエストをクリックします。
  3. Pull Requestで Files changed(変更されたファイル)をクリックしてください。

Pull Request Files changed tab

  1. Hover over the line of code where you'd like to add a comment, and click the blue comment icon. To add a comment on multiple lines, click and drag to select the range of lines, then click the blue comment icon. 青いコメントアイコン
  2. コメントフィールドにコメントを入力してください。 コメントフィールド
  3. Optionally, to suggest a specific change to the line or lines, click , then edit the text within the suggestion block. サジェッションブロック
  4. 完了したら、[Start a review] をクリックします。 レビューがすでに開始していた場合は、[Add review comment] (レビューコメントを追加) をクリックします。 [Start a review] ボタン

レビューを提出する前は、行のコメントは保留中であり、自分にしか見えません。 レビューを提出する前ならばいつでも、保留中のコメントを編集できます。 その保留中のコメントのすべてを含めて、保留中のレビューをキャンセルするには、[Conversation] タブでタイムラインの最後まで下にスクロールし、[Cancel review] をクリックします。

[Cancel review] ボタン

依存関係の変更をレビューする

プルリクエストに依存関係への変更が含まれている場合は、マニフェストまたはロックファイルの依存関係のレビューを使用して、何が変更されたかを確認し、変更によるセキュリティの脆弱性の発生の有無を確認できます。 詳しい情報については「Pull Request中の依存関係の変更のレビュー」を参照してください。

  1. Pull Requestで Files changed(変更されたファイル)をクリックしてください。

Pull Request Files changed tab

  1. マニフェストまたはロックファイルのヘッダの右側で、リッチ diff ボタンをクリックして依存関係のレビューを表示します。

    リッチ diff ボタン

  2. You may also want to review the source diff, because there could be changes to the manifest or lock file that don't change dependencies, or there could be dependencies that GitHub can't parse and which, as a result, don't appear in the dependency review.

    To return to the source diff view, click the button.

    ソース diff ボタン

ファイルをレビュー済みとしてマークする

ファイルのレビュー後は、そのファイルをレビュー済みとしてマークできます。マークしたファイルは折りたたまれます。 ファイルを表示後に変更すると、レビュー済みマークが解除されます。

  1. Pull Requestで Files changed(変更されたファイル)をクリックしてください。

Pull Request Files changed tab

  1. レビューを完了したファイルの、ヘッダの右側にある [Viewed] を選択します。 [Viewed] チェックボックス

レビューを提出する

プルリクエスト内でレビューしたいファイルをすべてレビューし終えたら、レビューをサブミットします。

  1. Pull Requestで Files changed(変更されたファイル)をクリックしてください。

Pull Request Files changed tab

  1. 変更されたコードの上で、Review changes(変更をレビュー)をクリックしてください。 変更のレビューボタン
  2. 提案された変更についてのフィードバックをまとめたコメントを入力してください。 レビューのまとめコメントウィンドウ
  3. 残しておくレビューの種類を選択します。 レビュー オプションを選択するラジオ ボタン
    • 変更を明確には承認せず、さらなる変更をリクエストすることもなく、おおまかなフィードバックだけを残したい場合は、[Comment] を選択します。
    • フィードバックを提出して、Pull Request で提案された変更をマージすることを承認するには、[Approve] を選択します。
    • Pull Request をマージする前に対処すべき問題をフィードバックするには、[Request changes] を選択します。
  4. Submit review(レビューをサブミット)をクリックしてください。

ヒント:

  • 必須レビューが有効化されており、リポジトリへの書き込み管理オーナーのいずれかの権限を持つコラボレータがレビューを要求する変更をサブミットすると、そのプルリクエストは同じコラボレータがプルリクエスト中の変更を承認する他のレビューをサブミットするまではマージできません。
  • リポジトリのオーナーと管理者は、プルリクエストが承認レビューを受けていなかったり、あるいは変更をリクエストしたレビュー担当者がOrganizationを離れていたりアクセスできなくなっている場合でも、プルリクエストをマージできます。
  • 必須レビューと古いレビューの棄却がどちらも有効化されており、承認済みのプルリクエストのブランチにコードを変更するコミットがプッシュされた場合、その承認は却下されます。 そのプルリクエストは、再度レビューされ承認されるまではマージできません。
  • 同じコミットを指す複数のオープンされたプルリクエストがあり、それぞれがheadブランチを持つ場合、いずれかがペンディングあるいは拒否されたレビューを持っているなら、それらはマージできません。

参考リンク

Did this doc help you?

Privacy policy

Help us make these docs great!

All GitHub docs are open source. See something that's wrong or unclear? Submit a pull request.

Make a contribution

OR, learn how to contribute.