Page MenuHomePhabricator

Merge the "--raw" and "--raw-command" flags to "arc diff" into "--raw <path>"
Needs ReviewPublic

Authored by epriestley on May 30 2020, 11:16 PM.
Tags
None
Referenced Files
F13057368: D21305.diff
Fri, Apr 19, 2:11 PM
Unknown Object (File)
Fri, Apr 5, 7:35 PM
Unknown Object (File)
Mar 3 2024, 2:00 PM
Unknown Object (File)
Feb 21 2024, 3:30 AM
Unknown Object (File)
Feb 13 2024, 10:07 AM
Unknown Object (File)
Feb 11 2024, 5:32 PM
Unknown Object (File)
Feb 2 2024, 9:24 AM
Unknown Object (File)
Jan 30 2024, 8:20 AM
Subscribers
None
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None
Maniphest Tasks
T13544: Modernize the "arc diff" workflow
Summary

Ref T13544. These flags can reasonably be combined into a single flag, and I'm also not entirely comfortable with executing arbitrary subprocesses directly from CLI arguments in modern arc. There are other ways to accomplish this goal, but I'd like to move toward fewer sources of arbitrary command execution.

Test Plan

Created this revision with git show | arc diff --raw.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 24544
Build 33828: arc lint + arc unit

Event Timeline

epriestley created this revision.