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

Reviewing proposed changes in a pull request

In a pull request, you can review and discuss commits, changed files, and the differences (or "diff") between the files in the base and compare branches.

About reviewing pull requests

You can review changes in a pull request one file at a time. While reviewing the files in a pull request, you can leave individual comments on specific changes. After you finish reviewing each file, you can mark the file as viewed. This collapses the file, helping you identify the files you still need to review. A progress bar in the pull request header shows the number of files you've viewed. After reviewing as many files as you want, you can approve the pull request or request additional changes by submitting your review with a summary comment.

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

Starting a review

  1. リポジトリ名の下で、クリックします Pull requests

    ![Pull request tab selection](/assets/images/enterprise/3.3/repository/repo-tabs-pull-requests.png)
    
  2. プルリクエストのリストで、レビューしたいプルリクエストをクリックします。

  3. プルリクエストで Files changed(変更されたファイル)をクリックしてください。 プルリクエストの変更されたファイルタブ

  4. コメントを追加したいコードの行の上にカーソルを移動し、青いコメントアイコンをクリックしてください。複数行にコメントを追加するには、クリックしてからドラッグで行の範囲を選択し、続いて青いコメントアイコンをクリックしてください。 青いコメントアイコン

  5. コメントフィールドにコメントを入力してください。 コメントフィールド

  6. あるいは、特定の変更を行あるいは複数行に対して示唆するには、をクリックし、示唆するブロック内のテキストを編集してください。 サジェッションブロック

  7. When you're done, click Start a review. If you have already started a review, you can click Add review comment.

    Start a review button

Before you submit your review, your line comments are pending and only visible to you. You can edit pending comments anytime before you submit your review. To cancel a pending review, including all of its pending comments, scroll down to the end of the timeline on the Conversation tab, then click Cancel review.

Cancel review button

Reviewing dependency changes

注釈: 依存関係のレビューは現在ベータであり、変更される可能性があります。

If the pull request contains changes to dependencies you can use the dependency review for a manifest or lock file to see what has changed and check whether the changes introduce security vulnerabilities. For more information, see "Reviewing dependency changes in a pull request."

  1. プルリクエストで Files changed(変更されたファイル)をクリックしてください。 プルリクエストの変更されたファイルタブ

  2. On the right of the header for a manifest or lock file, display the dependency review by clicking the rich diff button.

    The rich diff button

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

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

    ソース diff ボタン

Marking a file as viewed

After you finish reviewing a file, you can mark the file as viewed, and the file will collapse. If the file changes after you view the file, it will be unmarked as viewed.

  1. プルリクエストで Files changed(変更されたファイル)をクリックしてください。 プルリクエストの変更されたファイルタブ

  2. On the right of the header of the file you've finished reviewing, select Viewed.

    Viewed checkbox

Submitting your review

After you've finished reviewing all the files you want in the pull request, submit your review.

  1. プルリクエストで Files changed(変更されたファイル)をクリックしてください。 プルリクエストの変更されたファイルタブ

  2. 変更されたコードの上で、Review changes(変更をレビュー)をクリックしてください。 変更のレビューボタン

  3. 提案された変更についてのフィードバックをまとめたコメントを入力してください。 レビューのまとめコメントウィンドウ

  4. Select the type of review you'd like to leave:

    Radio buttons with review options

    • Select Comment to leave general feedback without explicitly approving the changes or requesting additional changes.
    • Select Approve to submit your feedback and approve merging the changes proposed in the pull request.
    • Select Request changes to submit feedback that must be addressed before the pull request can be merged.
  5. 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を承認することはできません。

Further reading

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

プライバシーポリシー

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

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

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

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

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