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
F15400054: D21325.id.diff
Mon, Mar 17, 9:22 AM
F15383144: D21325.id50755.diff
Fri, Mar 14, 3:39 PM
F15382670: D21325.id50789.diff
Fri, Mar 14, 1:46 PM
F15382654: D21325.diff
Fri, Mar 14, 1:38 PM
F15381731: D21325.id50755.diff
Fri, Mar 14, 9:06 AM
F15333855: D21325.id50755.diff
Sat, Mar 8, 5:12 AM
F15332425: D21325.diff
Fri, Mar 7, 5:24 PM
Unknown Object (File)
Feb 9 2025, 9:30 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.