Review and discuss a pull request

お困りですか?

アトラシアン コミュニティをご利用ください。

コミュニティに質問

The review phase of a pull request typically involves reviewers making comments and the author pushing additional changes and commenting in response, until the pull request is ultimately approved. The pull request author usually starts by adding colleagues as reviewers. Reviewers then leave comments–either on the entire pull request or on a specific part of the code changes–and then update their reviewer status to notify the author that they've completed their review. 

You can also create to-do lists that can act as a reminder to ensure that changes made to a pull request before merging are completed by creating a pull request task.

Depending on the feedback provided by reviewers, the author may then update a pull request with new commits. This may be to clean up the code, resolve any outstanding tasks, or improve the quality of the code. They can also apply suggestions to quickly make code changes for minor issues like typos.

Once the cycles of reviewer feedback and new commits has reached a conclusion, a pull request can either be merged to its target branch, or declined if the changes are not to be merged.


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

Bitbucket Server allows you to add one or more reviewers to a single pull request who can then approve, or decline the request. Pull requests give those who have access to the repository, the ability to review the quality of the code that’s specified in the pull request. 

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

  1. Access the pull request by either following links from an email notification, selecting a notification within the pull request inbox (in the upper-right), or searching for a pull request by clicking Pull requests on the sidebar (read more about searching for pull requests).
  2. プル リクエスト内で、チームメイトの変更やコメントをレビューします。 
  3. いずれかのビューで変更についてフィードバックを残し、同僚に直接質問するには @メンションを使用します。コメントを入力すると、メンションされたユーザーに通知が届きます。コメントを残すさまざまな方法 (プル リクエスト タスクを含む) の詳細については、「プル リクエストについてディスカッションする」セクションを参照してください。
  4. Finish your review by indicating if you feel the pull request can be merged, or if the author of the pull request needs to make additional changes before you can provide your stamp of approval. Select either the status indicators to let your team know you've reviewed the changes and the ball is now in their court:
    • Approve - indicates you've reviewed the changes and the code is ready to be merged.
    • Needs work - indicates you've reviewed the changes, but the code is not quite ready to be merged.

プル リクエストから Jira 課題を作成

Bitbucket Server allows you to create a Jira issue directly from a pull request comment or task instead of being forced to leave Bitbucket and return with a created issue.   

To create an issue from a pull request comment or task:

  1. Create a new issue by clicking ...Create Jira issue.
  2. [プロジェクト] と [課題タイプ] を選択します。
  3. Add a summary, description, and any other fields as needed.
  4. [課題の作成] をクリックします。

作成した課題が Jira 内で課題が削除された場合、それは引き続き課題として計上されますが、プル リクエストには表示されなくなります。

この機能は、プル リクエストの処理後に扱う必要があるスピンオフ課題を作成する場合に最適です。

プル リクエストの一部としてワークフロー アイテムを追加することは推奨されません。

プル リクエストについて議論する

プル リクエストの最も重要な側面は、そこで行われる議論ですプル リクエスト全体、特定のファイル、またはファイル内の特定のコード行にコメントすることができます。コメントへのいいね! を行うことで、"このフィードバックについても検討してください" というメッセージ効果的に伝え、レビューのフィードバックを素早く増幅させることができます。また、タスクを任意のコメントに関連付け、レビューの際に特定されたアクションを簡単に追跡および解決できるようにします。「プル リクエスト タスク」の詳細をご覧ください。

変更を確認する主な方法は 3 つあります。

  1. 概要タブ - プル リクエストの作成以降のプル リクエストのすべてのアクティビティが一覧表示されます。
  2. 差分タブ - 追加、削除、または変更されたコード行をハイライト表示します。
  3. コミット タブ - マージされるすべてのコミットを一覧表示します。クリックすると 1 つのコミットを表示できます。

Viewing other comments

When updating a pull request, comments that are in older diffs or that have become outdated due to a pull request update will become hidden. To view them, click the other comments counter button at the top of the page to open a dialog box with more context as to why code has changed throughout a pull request. You’ll be able to:

  • see a file's activity stream showing comments that are outdated or appear on another diff
  • 実際に古くなってしまっているコメントの区別
  • reply to, like, delete, or react to outdated comments the same way you can from the overview tab

概要

[概要] タブには、最初の作成から最後のマージ (または却下) までのプル リクエスト上のチームのすべてのアクティビティが一か所で表示され、途中のすべてのコメント、返信、およびコミットを確認できます。

You can add a comment on the Overview tab (just under 'Activity'), or reply to a previous comment. Use mentions to alert another Bitbucket Server user to your comment, and use Markdown to add formatting, for example headings or lists.


Diff ビュー

The diff view highlights the changes between the source and target branch. You can choose whether you want to highlight word level changes in the diff, or display details like comments and whitespace changes from the Diff view settings  icon.

The Unified diff option shows the changes in one continuous column.

The Side-by-side diff lets you easily compare the changes by showing the original file on one side, highlighting removed and modified lines, and then changed files on the other side, highlighting added and modified lines. 


インタラクティブなレビュー

Within the diff view, using the Change Scope selector, you can select a specific commit to review, or choose to view all changes within a pull request. If you return to a pull request that you previously reviewed, you'll only see the new commits added since you last reviewed the pull request.

変更を確認できる「コミット」

[コミット] タブには、マージされるすべてのコミットが一覧表示されます (灰色で表示されているものはマージ済みです)。コミットをクリックすると、プル リクエストのコンテキストで、プル リクエストの一部としてコミットをレビューできます。 

コミットを表示する際は、コミット内の任意のファイルに対し、ファイル全体、特定のコード行、diff に対してコメントを追加できます。

参加者はブランチに新しい変更をコミットできます。Bitbucket Server はプル リクエストの [コミット] タブを自動更新するため、どのコミットがマージされるかを正確に確認できます。行が追加または削除された場合、Bitbucket Server はそれに合わせてコメントを移動します。コメントのついた行が削除されてもアクティビティでコメントを表示できますが、最近のコミットで対象のコードが変更されたことがわかるよう、Bitbucket Server は diff を outdated としてマークします。

プル リクエスト内で 1 つのコミットを表示する

Pull request suggestions

As a reviewer of a pull request, you can suggest a small change to the code by leaving a suggestion right inside the comment or task itself. If you have write access to the source repository, you can commit the suggested change directly in the pull request without further action.

To create a pull request suggestion:

  1. In the comment dialog, click .

  2. Type your suggestion in the code block. You can also add any comments around it.

To apply a pull request suggestion:

  1. Click Apply suggestion.
  2. Edit the commit message if required and then click Commit changes to add a new commit to the pull request.

プル リクエスト タスク

You can create a task on the entire pull request, a particular fie, or on specific lines of code in a file to track required work identified during a code review. Convert a task into a comment, or a comment into a task. Anyone with permission to browse a pull request can resolve them. Repository admins and pull request authors can only edit, delete, or convert their own tasks. A Bitbucket Server administrator can enable a merge check that requires all tasks to be resolved before the pull request can be merged. See Checks for merging pull requests.

To create a pull request task, click Add a comment  or the  icon on a line of code in the diff view and add your text, then click Create task. Use Markdown to add formatting, images, and attachments to your tasks.

 To indicate a pull request task is done, tick the box next to the task.

To see all the unresolved tasks for a pull request, use Shift+T on the pull request page, or click the Open tasks list button. 

To see a task in the context of where it was created, in the Tasks window, click the 'View on' linked text.

Find matching code in a pull request

When you want to locate code in a pull request, you can search for it within changed files in the diff view.

To locate code within changed files, click Search code and enter your text into the search field. Only the files containing the match are then displayed expanded in the file tree with the number of occurrences highlighted per file. Easily navigate between occurrences by clicking through each line.

Find files in a pull request 

To filter through changed files, click Filter file tree and enter the file name into the field. You can search through files in folders using wildcards.

Some common wildcards supported are *, ?, and **. For example; filtering on **/test/**/*mock*, reveals all files or folders within a directory called test that uses mock in their name. Note that matching is case-insensitive, so the filter will match both text/one/my-mock-test.js and Test/two/MyMockTest.java.

ウォッチと通知

プル リクエストにレビュアーとして追加された場合、またはプル リクエストに関するアクション(コメントの追加など) を実行した場合は、プル リクエストのウォッチャーとして自動的に追加されます。

操作ウォッチャーになるかどうか
レビュアーとして追加された(tick)
レビュワーとして自分自身を追加する(tick)
プル リクエストにコメントした(tick)
コメントに返信した(tick)
ソース ブランチにプッシュした(tick)

You can manually add yourself as a watcher by selecting Watch from the selection menu drop-down on the pull request screen.

You can stop watching a pull request by clicking the link in the email notification, or ...Unwatch on the pull request screen. If you stop watching a pull request, you will not automatically be added as a watcher again if you subsequently perform an action that would otherwise have added you.

Bitbucket Server sends email notifications to watchers when certain pull request events occur. Email notifications are batched by default, but you can change your personal account settings (on the Notification settings tab) so that you get notifications immediately.  The following notifications however, are always sent immediately to:

  • the reviewers when a pull request is created
  • all watchers when a pull request is deleted
  • a user when they are added as a reviewer to a pull request
  • a user when they are removed as a reviewer from a pull request
  • a user when they are mentioned in the description of a pull request

詳細は、「通知」を参照してください。

最終更新日 2020 年 7 月 9 日

この内容はお役に立ちましたか?

はい
いいえ
この記事についてのフィードバックを送信する
Powered by Confluence and Scroll Viewport.