Phabricator/Differential

From MediaWiki.org
Jump to navigation Jump to search
How to get Wikidata code review done.jpg

Differential is the code review tool in Phabricator[1]. As per discussions[2][3] with the technical community the WMF Release Engineering Team is driving the migration to Differential from Gerrit. See the #Gerrit-Migration Phabricator workboard and project to follow the progress.

Getting Started[edit]

Install Arcanist[edit]

To submit patches/changes ("Differentials"), install Arcanist, the command line tool that interfaces with Differential.

Background reading[edit]

  1. Read these recommended user guides from the upstream documentation on the two main parts of Differential code review:
    1. Differential User Guide
    2. Arcanist User Guide
  2. Read these recommended guides from upstream documentation on how code review happens and why:
    1. Writing Reviewable Code
    2. Differential Test Plans
    3. Bonus points: Read this blog post from the lead developer of Phabricator about how and why Differential does code-review.

Done[edit]

Now, go contribute code!

How can I do this thing I did in Gerrit?[edit]

Firstly, not all actions that Gerrit provides will be available in Differential. This is by design. Not all code-review tools act exactly the same and design decisions are made to make the code-review tool internally consistent. Thus, please first try to understand how Differential and Arcanist are meant to be used (by reading the above links and trying things out) before asking for feature parity with Gerrit.

For code contributors / authors[edit]

Take over another author's change[edit]

There is the "Commandeer Revision" action on every open diff. This will, as the name suggests, allow you to take over a change. Do this when, for example, the original author is unresponsive but you want to move the change forward.

Amend another author's change[edit]

Instead of in Gerrit:

  git-review -d 12345 # this applies change 12345 to your checkout of the repo
  # make minor change to fix typo or rebase and fix minor conflict
  git commit --amend -a
  git-review -R
  # Optionally CR+2/merge that patch

In Differential you do (still 3 commands, minus whatever fixes you make):

  arc patch D12345 # this applies D12345's change to your checkout of the repo
  # make minor change to fix typo or rebase and fix minor conflict
  git commit --amend -a
  arc diff --update D12345 HEAD^
  # Optionally 'arc land' if you simply want to commit the change

For code reviewers[edit]

Get notifications about patches submitted to your project[edit]

If you are a maintainer, please consider configuring the Owners tool to send you an email for new patchsets in a project (repository). Owners allows you to create "packages". A package consists of one or more directories in one or more code repositories. When someone proposes changes which affect code in a package that you own, you can automatically get notified via the "Auto Review" setting. Additionally, packages can trigger audits which alert you to any commits which haven't been reviewed by one of the package owners. To learn more about the owners tool, read the upstream documentation. After that, please use this form to create packages covering the code you care about.

References[edit]