際際滷

際際滷Share a Scribd company logo

Phabricator Workflow

M
Michael Schuett

Phabricator is a suite of tools for code review and project management. It includes Differential for pull requests, Maniphest for issues tracking, and Arcanist as the command line interface. With Arcanist, developers can create features, diff code changes, and land changes after review. The review process is blocking in Phabricator and encourages active participation from reviewers and authors through comments, memes, and requested changes. Other tools like Herald and Audit allow tracking changes and reviewing commit histories.

1 of 12
Downloaded 26 times
Phabricator
Why? http://cramer.io/2014/05/03/on-pull-requests/
Features Arcanist (CLI) Herald (Stalking People) Differential (*Pull Request) Diffusion (Repo Browser) Maniphest (Issues) Audit (Commit Review)
Arcanist brew install HEAD arcanist brew reinstall HEAD arcanist Documentation
Arcansit (work鍖ow) git pull {url} arc feature {my-feature} arc diff *Review / Make Revisions / Approval* arc land
arc feature arc feature {my-feature} defaults to making a feature off of origin/master arc feature {change} origin/ {branch} upstream diverged, what do? git pull / git pull rebase differential will not show changes from rebase or merges. (Github?)
arc diff arc diff If you have not commit yet it will ask if you would like to add the staged changes to the diff. Answer a friendly questionnaire about why you havent tested your code yet. Confused what this is going to do? Run `arc which` and get an explanation.
Review You have just sent your change for review. Now what do you do? Go bug the person you have assigned to look at your diff. The review process in Phabricator is blocking and works best when everyone is actively involved. If you are reviewing you can request the diff author makes changes, accept the diff, or comment to get more info about something that is not clear. Make sure to post some memes. Authors love memes.
arc land arc land {feature-name} done arc land hold see what land will do without actually committing any changes to the remote branch you must git push if you run hold 鍖rst arc land onto {branch} This should have been con鍖gured properly based on the arc feature command you ran however.
Herald & Audit Herald allows you to keep track of those pesky interns. If you are not directly involved in a project but would like to keep up to date on it you can set any number of Herald rules to alert you of updates to a repo. This has a straight forward GUI that can be found on your phabricator install
Questions? StackOver鍖ow IRC #phabricator
More Info Other 際際滷s Installing Phabricator Ansible Installer Arcanist (non brewers)

More Related Content

Phabricator Workflow

  • 3. Features Arcanist (CLI) Herald (Stalking People) Differential (*Pull Request) Diffusion (Repo Browser) Maniphest (Issues) Audit (Commit Review)
  • 4. Arcanist brew install HEAD arcanist brew reinstall HEAD arcanist Documentation
  • 5. Arcansit (work鍖ow) git pull {url} arc feature {my-feature} arc diff *Review / Make Revisions / Approval* arc land
  • 6. arc feature arc feature {my-feature} defaults to making a feature off of origin/master arc feature {change} origin/ {branch} upstream diverged, what do? git pull / git pull rebase differential will not show changes from rebase or merges. (Github?)
  • 7. arc diff arc diff If you have not commit yet it will ask if you would like to add the staged changes to the diff. Answer a friendly questionnaire about why you havent tested your code yet. Confused what this is going to do? Run `arc which` and get an explanation.
  • 8. Review You have just sent your change for review. Now what do you do? Go bug the person you have assigned to look at your diff. The review process in Phabricator is blocking and works best when everyone is actively involved. If you are reviewing you can request the diff author makes changes, accept the diff, or comment to get more info about something that is not clear. Make sure to post some memes. Authors love memes.
  • 9. arc land arc land {feature-name} done arc land hold see what land will do without actually committing any changes to the remote branch you must git push if you run hold 鍖rst arc land onto {branch} This should have been con鍖gured properly based on the arc feature command you ran however.
  • 10. Herald & Audit Herald allows you to keep track of those pesky interns. If you are not directly involved in a project but would like to keep up to date on it you can set any number of Herald rules to alert you of updates to a repo. This has a straight forward GUI that can be found on your phabricator install
  • 12. More Info Other 際際滷s Installing Phabricator Ansible Installer Arcanist (non brewers)