Skip to content

Instantly share code, notes, and snippets.

@sekimura
Last active December 10, 2021 16:56
Show Gist options
  • Star 4 You must be signed in to star a gist
  • Fork 3 You must be signed in to fork a gist
  • Save sekimura/6367366 to your computer and use it in GitHub Desktop.
Save sekimura/6367366 to your computer and use it in GitHub Desktop.
phabricator workflow
  • create tasks T{NNNN} asign them
  • create a branch with name like "T{NNNN}-boo-hoo"
  • git checkout -b T1234-boo-foo
  • commit changes on that branch until it gets ready to be reviewed
  • git commit -am 'first'
  • git commit -am 'now it works'
  • check if it's lint free (NOTE: it runs lint against only modified files)
  • arc lint
  • push a review request to the server. This will create a diff with id D{NNNN}
  • arc diff
  • As a reviewer, you can apply the changeset on your local by using arc patch D{NNNN}
  • arc patch D5678
  • if reviewers post a comment and you need to update the changeset, commit more changes on that branch
  • and push updated changeset to the server
  • arc diff --update 1234
  • To check status of review requests which you have posted, run arc list which will give you a list of Status, Diff ID and title
  • arc list
  • Once review request got accepted, merge changes to master. "arc land" command will take care of merging/rebasing branch and deleting your working branch... And pushing changes to origin/master.
  • arc land
  • viola!
@brucenan
Copy link

If I just want to push code to branch, not master, arc land seems not works??

@lmr3796
Copy link

lmr3796 commented Jul 23, 2019

@brucenan Use arc diff --only

@iShawnWang
Copy link

iShawnWang commented Feb 8, 2020

Try arc land --onto TARGET_BRANCH @brucenan

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment