プルリクエスト ステップ2：レビュー

概要

この記事では、プルリクエストのレビュー手順（作成者と割り当てられたレビュアーの両方が関与する）を完了する方法を説明します。入念なレビューはコード品質を維持するために不可欠です。

ベストプラクティス

• 必ず少なくとも1人のレビュアーを追加して説明責任を確保してください。

• 説明フィールドを使ってレビュアー向けに主要な変更点を要約してください。

• すべての競合が解消され、テストが合格したときのみマージしてください。

• 使用する 変更を要求する 単に承認しない状態にする代わりに使用してください。ワークフローが明確に保たれます。

• 緊急対応を除き、自己レビューやレビュアーなしでのマージは避けてください。

プルリクエストの概要タブ

プルリクエストの詳細は、レビュアーやコントリビューターがリクエストの背景を理解するのに役立ちます。 概要 タブには確認すべき2つのセクションがあります： プルリクエストの詳細 および プルリクエストのレビュアー.

プルリクエストの詳細

フィールド	説明
ステータス	プルリクエストの現在の状態（作成済み, レビュー要求済み, 変更要求済み、および 承認済み)
ターゲットブランチ	変更がマージされるブランチです。
ソースブランチ	変更が発生した元のブランチです。
作成者	プルリクエストを作成したユーザー。
作成日	プルリクエストが最初に作成された日時。
更新日	プルリクエストに最後のアクティビティや変更があった日時。
説明	このプルリクエストでの変更点の要約。プルリクエストの作成者が記述します。

プルリクエストのレビュアー

この レビュアー セクションには、このプルリクエストのレビューを担当するユーザーが表示されます。各レビュアーのステータスは、（レビュー保留、変更要求、または承認）を示すアイコンで表示されます。レビュー手順の詳細は以下の指示を参照してください。

プルリクエストをレビューする

1

マージ競合チェックを待つ

1. プルリクエストが作成されると、最初にマージ競合のチェックが行われ、灰色のバナーで示されます。

   
2. バナーが次のいずれかに変わるのを待ってください：

   1. 緑: このブランチはマージ可能です

   2. オレンジ: このブランチは解決すべき競合があります

3. ブランチがマージ可能な場合は、この記事の指示に従ってプルリクエストのレビューを続行できます。

   
4. ブランチに競合がある場合は、このプルリクエストのレビューを続行する前に競合を解決する必要があります。競合の解決方法については、次の記事を参照してください。

   1. プルリクエストの競合の解決

2

（オプション）レビュアーを追加する

レビュアーを追加することでコード品質が確保され、早期に競合を発見でき、チームの承認プロセスが徹底されます。プルリクエストをマージする前に、すべてのレビュアーが必ずクリックする必要があります： 承認 。

自己レビュー

レビュアーが追加されていない場合、自分自身のプルリクエストの変更をレビューできます。正式なレビューや承認は必須ではありません。レビュー後、追加の承認なしでマージできます。

1. の レビュアー セクションで、 鉛筆 アイコンをクリックします。

   
2. クリックする 追加 してこのプルリクエストをレビューしてほしい各ユーザーを選びます。

   
3. クリックする 保存.

   
4. 追加されたレビュアーは レビュアー セクションに一覧表示され、プルリクエストのステータスは レビュー要求済み.

   

3

レビュー変更

レビューを行うには、各レビュアーが Cloud DevOps にログインして次の手順に従う必要があります：

レビュー待ちのプルリクエストはステータスで表示されます レビュー要求済み、および レビュアー' アイコンが表示されます。

自己レビュー

自己レビューを行う場合でも、マージ前に各コンポーネントを確認する必要があります。

1. クリックする プルリクエストは 左側のサイドバーのメニューから。

2. レビューするプルリクエストをクリックします。

   
3. クリックします 変更されたコンポーネント タブ。

4. レビューするコンポーネントをクリックします。

   

すべてのコンポーネントをレビューする必要があります。

1. コンポーネントを表示すると、ソースコードの ターゲット および ソース が差分をハイライトした状態で並べて表示されます。

   1. 赤: 削除される行。

   2. 緑: 追加される行。

   
2. 各コンポーネントについてレビュー手順を繰り返します。

4

（オプション）変更を要求する

レビュー時には、承認前に修正が必要であることを示すために 変更を要求する を使用してください。このステータスは、プルリクエスト作成者にソースブランチを必要な変更で更新するよう促します。要求が何を変更すべきかを特定していない場合は、必要な具体的な変更をチームに直接伝えてください。

自己レビュー

自己レビューを行っている場合、あなたはできません 変更を要求する.

1. クリックする 変更を要求する.

   
2. 変更を要求した後、プルリクエストのステータスは次のように変わります： 変更要求済み.

   

レビューの再提出

要求された変更を行った後、変更を要求したレビュアーに対してプルリクエストを再提出してください。

1. プルリクエストをクリックします。

   
2. クリックします 更新 を、レビューを要求したレビュアーの横にあるアイコンをクリックします。

   

5

承認

レビュアーがすべてのコンポーネントのレビューと、完了した変更要求の確認を終えたら、マージする前にプルリクエストを承認する必要があります。

すべてのレビュアーがクリックするまではプルリクエストは承認されません： 承認.

自己レビュー

自己レビュー時には、あなたはできません 承認 プルリクエストを承認する。ただしレビュアーがいないため、マージ前の承認は必要ありません。

1. クリックする 承認.

   
2. すべてのレビュアーが承認すると、プルリクエストのステータスは次のように変わります： 承認済み、そしてマージできます。

   

6

次の手順：マージ

プルリクエストが承認されると、ソースブランチの変更をターゲットブランチにマージする準備が整います。マージ手順については次の記事を参照してください。

• プルリクエスト ステップ3：マージ