Page MenuHomePhabricator

Improve the logic for identifying ambiguous commits and applying "--revision" to them
ClosedPublic

Authored by epriestley on Jun 7 2020, 3:53 PM.
Tags
None
Referenced Files
F15497392: D21325.diff
Sun, Apr 13, 10:22 AM
F15443914: D21325.id50789.diff
Thu, Mar 27, 6:53 AM
F15441136: D21325.id50755.diff
Wed, Mar 26, 4:39 PM
F15437757: D21325.id.diff
Tue, Mar 25, 9:18 PM
F15432044: D21325.id50755.diff
Mon, Mar 24, 3:56 PM
F15431163: D21325.diff
Mon, Mar 24, 10:55 AM
F15430088: D21325.id50789.diff
Mon, Mar 24, 5:37 AM
F15422733: D21325.id50755.diff
Sat, Mar 22, 10:23 AM
Subscribers
None

Details

Summary

Ref T13546. This is mostly minor cleanup that improves behavior under "--revision".

Test Plan

Ran arc land --into-empty and arc land --into-empty --revision 123 with ambiguous revisions in history to hit both the force and non-force outcomes.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Jun 8 2020, 11:22 PM
This revision was automatically updated to reflect the committed changes.