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
F18762114: D21325.id50789.diff
Mon, Oct 6, 5:38 PM
F18745720: D21325.id50755.diff
Fri, Oct 3, 8:57 AM
F18642598: D21325.diff
Fri, Sep 19, 12:16 AM
F18496216: D21325.diff
Sep 4 2025, 5:17 PM
F18477001: D21325.diff
Sep 3 2025, 1:47 AM
F18283865: D21325.diff
Aug 23 2025, 2:05 AM
F18198719: D21325.diff
Aug 18 2025, 1:10 AM
F18101161: D21325.id.diff
Aug 9 2025, 12:21 PM
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.