Page MenuHomePhabricator

Sometimes discard already-closed revisions in "arc land"
ClosedPublic

Authored by epriestley on Jun 5 2020, 11:18 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Nov 25, 11:45 AM
Unknown Object (File)
Thu, Nov 21, 5:53 AM
Unknown Object (File)
Sat, Nov 16, 11:34 PM
Unknown Object (File)
Oct 24 2024, 5:12 AM
Unknown Object (File)
Oct 15 2024, 2:25 PM
Unknown Object (File)
Oct 9 2024, 1:29 PM
Unknown Object (File)
Oct 9 2024, 1:29 PM
Unknown Object (File)
Oct 9 2024, 1:29 PM
Subscribers
None

Details

Summary

Ref T13546. When we find commits in history which are associated with already-closed revisions, and they weren't named explicitly on the command line, and we're using a squash strategy, discard them.

This generally happens when "feature2" is on top of "feature1", but "feature1" gets amended or branched elsewhere and lands independently.

Test Plan

Ran "arc land feature3" where prior revisions had already landed, got discards on the duplicated changes.

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:17 PM
This revision was automatically updated to reflect the committed changes.