This page is obsolete. It is being retained for archival purposes. It may document extensions or features that are obsolete and/or no longer supported. Do not rely on the information here being up-to-date.
The RfC Migrate code review and management to Phabricator from Gerrit was declined in March 2018, see T119908
Differential is the code review tool in Phabricator.
As per discussions with the technical community the WMF Release Engineering Team is driving the migration to Differential from Gerrit. See the #Gerrit-Migration Phabricator project.
To submit patches/changes ("Differentials"), install Arcanist, the command line tool that interfaces with Differential.
- Read these recommended user guides from the upstream documentation on the two main parts of Differential code review:
- Read these recommended guides from upstream documentation on how code review happens and why:
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.
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.
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 reviewersEdit
Get notifications about patches submitted to your projectEdit
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.
Since the migration to Differential has been scrapped in favor of continuing with Gerrit, we have been slowly removing Phabricator-hosted repositories. Some repositories, mostly those related to Phabricator or its extensions, are still managed with Differential, and you might find yourself in the rare situation where you need to submit a patch/commit to a Phabricator managed repository. If you do not want to go through the trouble of installing and configuring arcanist, then there is an alternative way to submit a change using the web interface. To do this you need three things:
- the name of the repository for which you want to submit a change,
- a local copy of the repository. You can just use a mirror, like those available in Gerrit, and
- the uncommitted changes locally.
- Optionally, you can put these in a diff file by running git diff > change.patch.
To submit a change, visit phab:/differential/diff/create/, which can also be found by navigating to Differential and then clicking on "Create Diff". There you can either paste your diff created by git diff or upload a diff file created by git diff > change.patch.
After clicking "Create Diff", you'll be shown a screen where you can input a summary, add tags, subscribers and reviewers, similar to creating a task.
Finally, after publishing the diff, don't forget to request a review by scrolling down the screen to the comment text box. There is a dropdown with the action "Request Review", which is a required action to get reviewers notified.
- Upstream documentation on Differential: https://secure.phabricator.com/book/phabricator/article/differential/
- The Migrate code review and management to Phabricator from Gerrit RFC.
- Discussion at WikiDev16 - notes and follow-ups.
- T119908 [RfC]: Migrate code review / management from Gerrit to Phabricator