Page MenuHomePhabricator
Differential D21074

Trivially update "arc branch/feature" and "arc browse" for Toolsets
ClosedPublic
Actions

Authored by epriestley on Apr 8 2020, 7:28 PM.
Tags
None
Referenced Files
F15464794: D21074.id50209.diff
Wed, Apr 2, 12:48 PM
F15456349: D21074.diff
Sun, Mar 30, 8:59 AM
F15443868: D21074.id50203.diff
Thu, Mar 27, 6:38 AM
F15442225: D21074.id50209.diff
Wed, Mar 26, 10:13 PM
F15439353: D21074.id50209.diff
Wed, Mar 26, 7:09 AM
F15437134: D21074.id.diff
Tue, Mar 25, 5:48 PM
F15432584: D21074.diff
Mon, Mar 24, 6:39 PM
F15430229: D21074.id50209.diff
Mon, Mar 24, 6:36 AM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T11968: Decide the fate of FutureGraph
T13490: Upgrade all "classic" Arcanist workflows to Toolsets
Commits
rARC391c1643134a: Trivially update "arc branch/feature" and "arc browse" for Toolsets
Summary

Ref T11968. Ref T13490. These are the workflows which currently use the intermediate-level hardpoint code (which made hardpoints formal, but didn't do the yield stuff).

Move toward updating them by doing some basic bookkeeping, with a few compatibility adjustments to the parent Workflow class.

Test Plan

Ran "arc branch" and "arc browse" with various arguments.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Apr 8 2020, 7:28 PM
Harbormaster completed remote builds in B24021: Diff 50203.Apr 8 2020, 7:29 PM
epriestley requested review of this revision.Apr 8 2020, 7:29 PM
epriestley added a child revision: D21075: Rename "getWorkingCopy()" to "getWorkingCopyIdentity()" in Arcanist.Apr 8 2020, 7:37 PM
This revision was not accepted when it landed; it landed in state Needs Review.Apr 10 2020, 11:23 AM
This revision was landed with ongoing or failed builds.
Closed by commit rARC391c1643134a: Trivially update "arc branch/feature" and "arc browse" for Toolsets (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rARC391c1643134a: Trivially update "arc branch/feature" and "arc browse" for Toolsets.

Revision Contents
Changeset List

Diff 50209

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/browse/workflow/ArcanistBrowseWorkflow.php

Loading...
View Options

src/workflow/ArcanistFeatureWorkflow.php

Loading...
View Options

src/workflow/ArcanistWorkflow.php

Loading...
Phabricator · Built by Phacility