Page MenuHomePhabricator

Make arc feature much more fancy and fun
Open, Needs TriagePublic

"Like" token, awarded by Pawka."Pirate Logo" token, awarded by yelirekim."Like" token, awarded by siepkes."Like" token, awarded by venky."Party Time" token, awarded by epriestley."Dat Boi" token, awarded by michaeljs1990.
Assigned To
Authored By
yelirekim, Aug 23 2016


We have an internal workflow called arc flow that does exactly what arc feature does, plus a bunch more useful things, like:

It shows us:

  • Yellow hashes for diffs which are different locally from the remote
  • Number of commits ahead and behind that something is from its parent branch
  • Actually render nested branches
  • Show abbreviated git status underneath everything

Nearly every dev in our building runs this command constantly.

Event Timeline

chad added a subscriber: chad.Aug 23 2016, 5:53 PM

which devs don't? it's the designers, isn't it.

avivey added a subscriber: avivey.Aug 23 2016, 5:59 PM

I'd call this thing arc status, because arc feature is "a wrapper on git branch", and flow makes me think of git-flow.

wait, by "remote" you're talking about remote branch or Revision?

Basically "is the diff people are reviewing identical to your local branch"


👍 that's what I was hoping for :)

venky awarded a token.Aug 24 2016, 9:31 PM
yelirekim assigned this task to jcox.EditedAug 25 2016, 4:01 PM
In T11518#191391, @chad wrote:

which devs don't? it's the designers, isn't it.

me code pretty one day

I'd call this thing arc status, because arc feature is "a wrapper on git branch", and flow makes me think of git-flow.

The line of thinking here was that arc feature with no flags already shows you status, it's just that that status could be much more useful. I'm not really too concerned about what this would end up being called though.

arc flow is poking fun at git flow because it is kindof a "workflow" just far less involved in complexity than git flow is, which was a concept people were talking about using right around the time I initially wrote this.

Can you share the code? :)

It relies on a server side API call so you'll have to actually extend your server install in order for it to work.

We can post the frankenstien of classes that underly this soon though, sure.

Maybe @jcox a good place to start would be creating two libphutil packages that can cleanly contain this? I'm not sure how deep into our homerolled infra this gets, at the very least we have classes for rendering trees / UTF16 character display, among other things to make this work.

I think this is all generally reasonable to bring upstream, at least in spirit, but I want to wait for planned arc changes (overviewed in T11429) to get a little further along and build it on top of that.

We should also be providing some of the data this needs access to to workflows in a better way ("which revision" in T11355, modern APIs in T10945).

Yomi added a subscriber: Yomi.Sep 29 2016, 6:13 PM
urzds added a subscriber: urzds.Nov 22 2016, 10:14 AM
eadler added a project: Restricted Project.Nov 23 2016, 12:56 PM
Pawka awarded a token.May 24 2017, 2:31 PM
Pawka added a subscriber: Pawka.