プルリクエストのレビューについて
プルリクエストの変更は、1 ファイルごとにレビューできます。 プルリクエストでファイルを確認しているときに、特定の変更について個別のコメントを残すことができます。 各ファイルの確認が終了したら、ファイルを閲覧済みとしてマークできます。 これによりファイルが折りたたまれるので、ま� レビューを必要とするファイルを特定するのに役立ちます。 プルリクエストヘッダのプログレスバーには、閲覧したファイル数が表示されます。 必要な数のファイルを確認した後、要約コメントを付けて確認を送信することにより、プルリクエストを承認するか、追� の変更をリクエストできます。
Tip: あなた、もしくはあなたがメンバーになっているTeamがレビューを求められているプルリクエストは、review-requested:[USERNAME]
あるいはteam-review-requested:[TEAMNAME]
という検索条件で検索できます。 詳しい情� �については「Issueやプルリクエストの検索」を参照してく� さい。
レビューを開始する
-
リポジトリ名の下で、クリックします Pull requests
-
プルリクエストのリストで、レビューしたいプルリクエストをクリックします。
-
プルリクエストで Files changed(変更されたファイル)をクリックしてく� さい。
-
コメントを追� したいコードの行の上にカーソルを移動し、青いコメントアイコンをクリックしてく� さい。複数行にコメントを追� するには、クリックしてからドラッグで行の範囲を選択し、続いて青いコメントアイコンをクリックしてく� さい。
-
コメントフィールドにコメントを入力してく� さい。
-
あるいは、特定の変更を行あるいは複数行に対して示唆するには、をクリックし、示唆するブロック内のテキストを編集してく� さい。
-
完了したら、[Start a review] をクリックします。 レビューがすでに開始していた� �合は、[Add review comment]� (レビューコメントを追� ) をクリックします。
レビューを提出する前は、行のコメントは保留中であり、自分にしか見えません。 レビューを提出する前ならばいつでも、保留中のコメントを編集できます。 その保留中のコメントのすべてを含めて、保留中のレビューをキャンセルするには、[Conversation] タブでタイ� ラインの最後まで下にスクロールし、[Cancel review] をクリックします。
ファイルをレビュー済みとしてマークする
ファイルのレビュー後は、そのファイルをレビュー済みとしてマークできます。マークしたファイルは折りたたまれます。 ファイルを表示後に変更すると、レビュー済みマークが解除されます。
-
プルリクエストで Files changed(変更されたファイル)をクリックしてく� さい。
-
レビューを完了したファイルの、ヘッダの右側にある [Viewed] を選択します。
レビューを提出する
プルリクエスト内でレビューしたいファイルをすべてレビューし終えたら、レビューをサブミットします。
-
プルリクエストで Files changed(変更されたファイル)をクリックしてく� さい。
-
変更されたコードの上で、Review changes(変更をレビュー)をクリックしてく� さい。
-
提案された変更についてのフィードバックをまとめたコメントを入力してく� さい。
-
残しておくレビューの種類を選択します:
- 変更を明確には承認せず、さらなる変更をリクエストすることもなく、おおまかなフィードバック� けを残したい� �合は、[Comment] を選択します。
- フィードバックを提出して、Pull Request で提案された変更をマージすることを承認するには、[Approve] を選択します。
- Pull Request をマージする前に対処すべき問題をフィードバックするには、[Request changes] を選択します。
-
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を承認することはできません。