Page MenuHomePhabricator
Differential D16933

Implement a revision resolver for `arc browse <commit>`
ClosedPublic
Actions

Authored by epriestley on Nov 23 2016, 7:50 PM.
Tags
None
Referenced Files
F22189377: D16933.diff
Tue, Aug 25, 11:18 AM
Unknown Object (File)
Sat, Aug 22, 3:53 AM
Unknown Object (File)
Jul 2 2026, 3:27 PM
Unknown Object (File)
Jun 30 2026, 9:57 AM
Unknown Object (File)
Jun 25 2026, 12:58 AM
Unknown Object (File)
Jun 22 2026, 12:54 AM
Unknown Object (File)
Apr 15 2026, 3:45 PM
Unknown Object (File)
Apr 7 2026, 7:36 PM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T10895: Support`arc browse --revision <commit>` and make `arc browse` with no arguments mean `... --revision HEAD`
Commits
rARC84857e48900e: Implement a revision resolver for `arc browse <commit>`
Summary

Ref T10895. This allows arc browse <commit name> to resolve to a revision, if an associated open revision exists.

Test Plan

Ran arc browse with various arguments.

Diff Detail

Repository
rARC Arcanist
Branch
xp6
Lint
Lint Passed
SeverityLocationCodeMessage
Advicesrc/ref/ArcanistRevisionRef.php:32XHP16TODO Comment
Unit
Tests Passed
Build Status
Buildable 14646
Build 19127: Run Core Tests
Build 19126: arc lint + arc unit

Event Timeline

epriestley updated this revision to Diff 40757.Nov 23 2016, 7:50 PM
epriestley retitled this revision from to Implement a revision resolver for `arc browse <commit>`.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T10895: Support`arc browse --revision <commit>` and make `arc browse` with no arguments mean `... --revision HEAD`.
epriestley mentioned this in T10895: Support`arc browse --revision <commit>` and make `arc browse` with no arguments mean `... --revision HEAD`.Nov 23 2016, 8:07 PM
chad accepted this revision.Nov 23 2016, 8:57 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Nov 23 2016, 8:57 PM
Closed by commit rARC84857e48900e: Implement a revision resolver for `arc browse <commit>` (authored by epriestley, committed by epriestley). · Explain WhyNov 23 2016, 9:27 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 40757

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/browse/loader/ArcanistBrowseCommitHardpointLoader.php

Loading...
View Options

src/browse/loader/ArcanistBrowseCommitURIHardpointLoader.php

Loading...
View Options

src/browse/loader/ArcanistBrowseRevisionURIHardpointLoader.php

Loading...
View Options

src/browse/workflow/ArcanistBrowseWorkflow.php

Loading...
View Options

src/loader/ArcanistGitCommitMessageHardpointLoader.php

Loading...
View Options

src/loader/ArcanistGitRevisionHardpointLoader.php

Loading...
View Options

src/ref/ArcanistRevisionRef.php

Loading...
Phabricator · Built by Phacility