Page MenuHomePhabricator

Rebuild "arc branch" on new "hardpoint" infrastructure
ClosedPublic

Authored by epriestley on Nov 14 2016, 10:16 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Oct 1, 4:10 AM
Unknown Object (File)
Tue, Oct 1, 4:09 AM
Unknown Object (File)
Tue, Oct 1, 4:09 AM
Unknown Object (File)
Tue, Oct 1, 3:56 AM
Unknown Object (File)
Tue, Oct 1, 3:02 AM
Unknown Object (File)
Sat, Sep 21, 12:19 AM
Unknown Object (File)
Thu, Sep 12, 8:52 AM
Unknown Object (File)
Thu, Sep 12, 8:52 AM
Subscribers
None
Tokens
"Party Time" token, awarded by yelirekim.

Details

Summary

Ref T11355. Ref T10895. Ref T11518. This is heading to experiemntal. This may or may not be a good idea, but basically it's a more generic version of Query classes in Phabricator.

This starts creating generic objects ("CommitRef", "BranchRef") which have attachable properties, like many Phabricator objects do. Here, they're formalized (and theoretically extensible), as "hardpoints".

So: a hardpoint is something on an object which you can attach stuff to, but which we don't start with the data for.

All of the logic for actually figuruing out how to attach stuff to hardpoints is also modular. Loader classes have code for loading stuff onto objects. For example, ArcanistMercurialBranchCommitHardpointLoader knows how to run hg log to build the commit for a branch.

One issue is that arc feature in Mercurial is 100% bookmarks, so maybe I should actually be making ArcanistRefRef here. But we can probbbably deal with that later.

This moves us somewhat closer to T11355 and T11518, although the immediate thing I want to do with it is define an ArcanistObjectNameRef and use hardpoints to load URIs for it for T10895.

Overall, I expect this will see some revision in future changes, and perhaps most of it will go away.

Test Plan

Ran arc branch / arc feature in Git and Mercurial repositories.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Rebuild "arc branch" on new "hardpoint" infrastructure.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)

Not sold on this being any good yet.

src/workflow/ArcanistFeatureWorkflow.php
179

This is like 1K lines of code but I was able to remove one instanceof so that's progress right?

chad edited edge metadata.
This revision is now accepted and ready to land.Nov 23 2016, 1:40 AM
This revision was automatically updated to reflect the committed changes.