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

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

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

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

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

レビューを開始する

  1. リポジトリ名の下で、クリックします Pull requests
    ![Issue とプルリクエストのタブの選択](/assets/images/help/repository/repo-tabs-pull-requests.png)
    
  2. プルリクエストのリストで、レビューしたいプルリクエストをクリックします。
  3. Pull Requestで Files changed(変更されたファイル)をクリックしてください。

Pull Request Files changed tab

  1. コメントを追加したいコードの行の上にカーソルを移動し、青いコメントアイコンをクリックしてください。複数行にコメントを追加するには、クリックしてからドラッグで行の範囲を選択し、続いて青いコメントアイコンをクリックしてください。 青いコメントアイコン
  2. コメントフィールドにコメントを入力してください。 コメントフィールド
  3. あるいは、特定の変更を行あるいは複数行に対して示唆するには、をクリックし、示唆するブロック内のテキストを編集してください。 サジェッションブロック
  4. 完了したら、[Start a review] をクリックします。 レビューがすでに開始していた場合は、[Add review comment] (レビューコメントを追加) をクリックします。 [Start a review] ボタン

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

[Cancel review] ボタン

You can use Codespaces to test, run, and review pull requests.

  1. Open the pull request in a codespace, as described in "Opening a pull request."
  2. In the Activity Bar, click the GitHub Pull Request view. This view only appears when you open a pull request in a codespace. Option to open PR in a codespace
  3. To review a specific file, click the Open File icon in the Side Bar. Option to open PR in a codespace
  4. To add review comments, click the + icon next to the line number. Type your review comment and then click Start Review. Option to open PR in a codespace
  5. When you are finished adding review comments, from the Side Bar you can choose to either submit the comments, approve the changes, or request changes. Option to open PR in a codespace

For more information on reviewing pull requests in Codespaces, see "Using Codespaces for pull requests."

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

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

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

Pull Request Files changed tab

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

    リッチ diff ボタン

  2. 依存関係を変化させないような変更がマニフェストあるいはロックファイルに行われているかもしれず、あるいはGitHubがパースできず、その結果として依存関係レビューに現れてこない依存関係があるかもしれないので、ソースdiffをレビューしたいこともあるでしょう。

    ソースdiffのビューに戻るには、ボタンをクリックしてください。

    ソース 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ブランチを持つ場合、いずれかがペンディングあるいは拒否されたレビューを持っているなら、それらはマージできません。
  • If your repository requires approving reviews from people with write or admin permissions, then any approvals from people with these permissions are denoted with a green check mark, and approvals from people without these permissions have a gray check mark. Approvals with a gray check mark do not affect whether the pull request can be merged.
  • Pull Requestの作者は、自分自身のPull Requestを承認することはできません。

参考リンク

このドキュメントは役立ちましたか?

プライバシーポリシー

これらのドキュメントを素晴らしいものにするのを手伝ってください!

GitHubのすべてのドキュメントはオープンソースです。間違っていたり、はっきりしないところがありましたか?Pull Requestをお送りください。

コントリビューションを行う

OR, コントリビューションの方法を学んでください。

問題がまだ解決していませんか?