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)
Wed, Apr 24, 11:54 PM
Unknown Object (File)
Sat, Apr 20, 5:18 PM
Unknown Object (File)
Thu, Apr 11, 8:11 AM
Unknown Object (File)
Fri, Mar 29, 1:25 PM
Unknown Object (File)
Feb 22 2024, 7:11 PM
Unknown Object (File)
Feb 22 2024, 7:07 PM
Unknown Object (File)
Feb 22 2024, 12:25 PM
Unknown Object (File)
Feb 14 2024, 12:37 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
Branch
land12
Lint
Lint Passed
SeverityLocationCodeMessage
Advicesrc/land/engine/ArcanistLandEngine.php:1486XHP16TODO Comment
Unit
Tests Passed
Build Status
Buildable 24572
Build 33873: Run Core Tests
Build 33872: arc lint + arc unit

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.