Page MenuHomePhabricator

Refine "arc land" behavior when pushing "onto" a new branch
ClosedPublic

Authored by epriestley on Jun 8 2020, 11:56 PM.
Tags
None
Referenced Files
F19914227: D21341.id50804.diff
Fri, Apr 3, 9:00 AM
F19913140: D21341.id50804.diff
Fri, Apr 3, 12:23 AM
F19863747: D21341.id.diff
Sat, Mar 14, 12:31 PM
F19812177: D21341.diff
Mar 4 2026, 3:57 AM
F19512918: D21341.id.diff
Jan 13 2026, 4:56 AM
F18978881: D21341.id.diff
Nov 16 2025, 8:56 PM
F18978200: D21341.diff
Nov 16 2025, 7:29 PM
F18916251: D21341.id.diff
Nov 9 2025, 8:44 AM
Subscribers
None

Details

Summary

Ref T13546. If the "onto" branch doesn't exist yet and has a "/" in it, we need to preface it with "refs/heads" explicitly.

Fix a "false/null" issue with argument validation.

Possibly, "arc land" should prompt you before creating branches in the remote.

Test Plan

Ran "arc land --onto does-not-exist/1.1" and created a branch.

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:56 PM
epriestley requested review of this revision.
This revision was automatically updated to reflect the committed changes.