Page MenuHomePhabricator

Remove "arc diff --only"
ClosedPublic

Authored by epriestley on Oct 27 2017, 4:20 PM.
Tags
None
Referenced Files
F18321394: D18739.diff
Sun, Aug 24, 4:53 PM
F18232829: D18739.diff
Wed, Aug 20, 4:04 PM
F18047560: D18739.id.diff
Sun, Aug 3, 12:30 PM
F17968680: D18739.diff
Aug 1 2025, 1:49 PM
F17849683: D18739.id.diff
Jul 27 2025, 8:59 AM
F17846737: D18739.id44988.diff
Jul 27 2025, 5:16 AM
F17841370: D18739.id44976.diff
Jul 26 2025, 10:20 PM
F17687088: D18739.diff
Jul 14 2025, 7:04 AM
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