HomePhabricator

Remove "arc diff --only"

Authored by epriestley on Oct 27 2017, 4:17 PM.

Description

Remove "arc diff --only"

Summary:
Experimental branch. Depends on D18738. Ref T2543.

To prepare for adding --draft, clean up some of the flags behaviors.

--only currently means --preview --nolint --nounit. I'm going to:

  • Remove --only.
  • Rename --preview to --only, since I think --only is a better flag for this behavior than --preview.
  • If you want the old --only, you can arc alias it to arc diff --only --nolint --nounit --browse or similar.

Test Plan: Grepped for 'only', only, etc.

Reviewers: amckinley

Reviewed By: amckinley

Maniphest Tasks: T2543

Differential Revision: https://secure.phabricator.com/D18739

Details

Committed
epriestleyOct 30 2017, 10:17 PM
Pushed
epriestleyOct 30 2017, 10:17 PM
Reviewer
amckinley
Differential Revision
D18739: Remove "arc diff --only"
Parents
rARCccd648b4c408: Remove "arc diff --plan-changes"
Branches
Unknown
Tags
Unknown
Tasks
T2543: Add a formal "Draft" / "Not Yet Ready for Review" state to Differential
Build Status
Buildable 18768
Build 25293: Run Core Tests