Page MenuHomePhabricator

Remove "arc diff --only"
ClosedPublic

Authored by epriestley on Oct 27 2017, 4:20 PM.
Tags
None
Referenced Files
F15456080: D18739.diff
Sun, Mar 30, 6:56 AM
F15441132: D18739.id44976.diff
Wed, Mar 26, 4:35 PM
F15430355: D18739.id44988.diff
Mon, Mar 24, 7:15 AM
F15429247: D18739.diff
Mon, Mar 24, 1:20 AM
F15420758: D18739.id44976.diff
Fri, Mar 21, 4:50 PM
F15356632: D18739.id44976.diff
Tue, Mar 11, 6:22 AM
F15333939: D18739.diff
Sat, Mar 8, 5:47 AM
Unknown Object (File)
Sun, Mar 2, 1:37 PM
Subscribers
None

Details

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.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable