Page MenuHomePhabricator

Remove "arc diff --only"

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



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.

Diff Detail

rARC Arcanist
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.Oct 27 2017, 4:20 PM
epriestley requested review of this revision.Oct 27 2017, 4:21 PM
amckinley accepted this revision.Oct 28 2017, 3:29 PM
This revision is now accepted and ready to land.Oct 28 2017, 3:29 PM
This revision was automatically updated to reflect the committed changes.