Page MenuHomePhabricator
Differential D21330

Format "arc land" passthru commands more nicely, and execute them from CWD
ClosedPublic
Actions

Authored by epriestley on Jun 7 2020, 8:12 PM.
Tags
None
Referenced Files
F14883301: D21330.id50760.diff
Sun, Feb 9, 1:14 AM
Unknown Object (File)
Mon, Feb 3, 2:21 AM
Unknown Object (File)
Mon, Feb 3, 12:10 AM
Unknown Object (File)
Wed, Jan 29, 11:42 AM
Unknown Object (File)
Mon, Jan 27, 5:20 PM
Unknown Object (File)
Mon, Jan 27, 10:23 AM
Unknown Object (File)
Sat, Jan 25, 8:07 AM
Unknown Object (File)
Sat, Jan 25, 8:06 AM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13380: `git push` fatals when CWD does not exist
T13546: Modernize the "arc land" workflow
Commits
rARCf3f31155b761: Format "arc land" passthru commands more nicely, and execute them from CWD
Summary

Fixes T13380. Ref T13546. Use slightly nicer command formatting for passthru commands, to make it more obvious what's going on and which pieces of output are from "arc" vs various subcommands.

Also, execute repository API passthru commands from the working copy root. All other commands already did this, the older API just didn't support it.

Test Plan

Ran "arc land" in Git and Mercurial repositories, saw nicer output formatting.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Jun 7 2020, 8:12 PM
Harbormaster completed remote builds in B24582: Diff 50760.Jun 7 2020, 8:12 PM
epriestley requested review of this revision.Jun 7 2020, 8:12 PM
This revision was not accepted when it landed; it landed in state Needs Review.Jun 8 2020, 11:26 PM
Closed by commit rARCf3f31155b761: Format "arc land" passthru commands more nicely, and execute them from CWD (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rARCf3f31155b761: Format "arc land" passthru commands more nicely, and execute them from CWD.

Revision Contents
Changeset List

Diff 50794

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/api/ArcanistGitAPI.php

Loading...
View Options

src/repository/api/ArcanistMercurialAPI.php

Loading...
View Options

src/repository/api/ArcanistRepositoryAPI.php

Loading...
View Options

src/toolset/command/ArcanistCommand.php

Loading...
Phabricator · Built by Phacility