Paths

Table of Contentst

Differential D21325

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

Authored by epriestley on Jun 7 2020, 3:53 PM.

Tags

None

Referenced Files

	F21411027: D21325.diff
	Wed, Jul 29, 1:01 PM

	Unknown Object (File)
	Wed, Jul 1, 2:02 AM

	Unknown Object (File)
	Wed, Jul 1, 2:02 AM

	Unknown Object (File)
	Jun 18 2026, 12:09 AM

	Unknown Object (File)
	Apr 22 2026, 9:46 AM

	Unknown Object (File)
	Mar 5 2026, 5:01 AM

	Unknown Object (File)
	Dec 24 2025, 6:56 AM

	Unknown Object (File)
	Dec 11 2025, 9:43 AM

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13546: Modernize the "arc land" workflow
Commits: rARC709c9cb6fbe8: Improve the logic for identifying ambiguous commits and applying "--revision"…

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

Branch

land18

Lint

Lint Passed

Severity	Location	Code	Message
Advice	src/land/engine/ArcanistLandEngine.php:907	XHP16	TODO Comment
Advice	src/land/engine/ArcanistLandEngine.php:947	XHP16	TODO Comment
Advice	src/land/engine/ArcanistLandEngine.php:950	XHP16	TODO Comment
Advice	src/land/engine/ArcanistLandEngine.php:952	XHP16	TODO Comment

Unit

Tests Passed

Build Status

Buildable 24577
Build 33880: Run Core Tests
Build 33879: arc lint + arc unit

Event Timeline

epriestley created this revision.Jun 7 2020, 3:53 PM

Harbormaster completed remote builds in B24577: Diff 50755.Jun 7 2020, 3:53 PM

epriestley requested review of this revision.Jun 7 2020, 3:53 PM

This revision was not accepted when it landed; it landed in state Needs Review.Jun 8 2020, 11:22 PM

Closed by commit rARC709c9cb6fbe8: Improve the logic for identifying ambiguous commits and applying "--revision"… (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rARC709c9cb6fbe8: Improve the logic for identifying ambiguous commits and applying "--revision"….

Revision Contents
Changeset List

Path

Size

src/

land/

ArcanistLandCommit.php

10 lines

engine/

ArcanistLandEngine.php

179 lines

Diff 50755

src/land/ArcanistLandCommit.php

Loading...

src/land/engine/ArcanistLandEngine.php

Loading...