Page MenuHomePhabricator
Differential D21354

In "arc land" under Git, confirm branch creation
ClosedPublic
Actions

Authored by epriestley on Jun 11 2020, 2:48 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 24, 11:50 PM
Unknown Object (File)
Sat, Apr 20, 5:22 PM
Unknown Object (File)
Thu, Apr 11, 8:05 AM
Unknown Object (File)
Sun, Apr 7, 11:45 AM
Unknown Object (File)
Tue, Apr 2, 5:00 AM
Unknown Object (File)
Fri, Mar 29, 8:22 PM
Unknown Object (File)
Thu, Mar 28, 9:45 AM
Unknown Object (File)
Feb 19 2024, 12:40 PM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13546: Modernize the "arc land" workflow
Commits
rARCb0a9ef8351f1: In "arc land" under Git, confirm branch creation
Summary

Ref T13546. If "arc land" would create a branch, warn the user before it does.

Test Plan

Ran "arc land --onto mtarse", a typo of "master".

Diff Detail

Repository
rARC Arcanist
Branch
landx7
Lint
Lint Errors
SeverityLocationCodeMessage
Errorsrc/repository/marker/ArcanistGitRepositoryMarkerQuery.php:173XHP5Use of Undeclared Variable
Unit
Tests Passed
Build Status
Buildable 24651
Build 33987: Run Core Tests
Build 33986: arc lint + arc unit

Event Timeline

epriestley created this revision.Jun 11 2020, 2:48 PM
Harbormaster completed remote builds in B24651: Diff 50831.Jun 11 2020, 2:48 PM
epriestley requested review of this revision.Jun 11 2020, 2:48 PM
epriestley updated this revision to Diff 50832.Jun 11 2020, 2:48 PM
  • Fix undeclared "$markers".
Harbormaster completed remote builds in B24652: Diff 50832.Jun 11 2020, 2:49 PM
This revision was not accepted when it landed; it landed in state Needs Review.Jun 30 2020, 1:29 PM
Closed by commit rARCb0a9ef8351f1: In "arc land" under Git, confirm branch creation (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rARCb0a9ef8351f1: In "arc land" under Git, confirm branch creation.

Revision Contents
Changeset List

PathSize
src/
land/
engine/
61 lines
5 lines
18 lines
repository/
marker/
53 lines
xsprintf/
1 line

Diff 50831

View Options

src/land/engine/ArcanistGitLandEngine.php

Loading...
View Options

src/land/engine/ArcanistLandEngine.php

Loading...
View Options

src/land/engine/ArcanistMercurialLandEngine.php

Loading...
View Options

src/repository/marker/ArcanistGitRepositoryMarkerQuery.php

Loading...
View Options

src/xsprintf/tsprintf.php

Loading...
Phabricator · Built by Phacility