Page MenuHomePhabricator

Rename "arc diff --preview" to "arc diff --only"
ClosedPublic

Authored by epriestley on Oct 27 2017, 4:25 PM.
Tags
None
Referenced Files
F15458477: D18740.id44979.diff
Mon, Mar 31, 3:59 AM
F15458476: D18740.id44989.diff
Mon, Mar 31, 3:59 AM
F15458473: D18740.id.diff
Mon, Mar 31, 3:59 AM
F15443136: D18740.id44989.diff
Thu, Mar 27, 2:38 AM
F15435116: D18740.id.diff
Tue, Mar 25, 6:51 AM
F15431595: D18740.diff
Mon, Mar 24, 1:21 PM
F15431172: D18740.id.diff
Mon, Mar 24, 10:57 AM
F15387378: D18740.diff
Sat, Mar 15, 2:18 AM
Subscribers
None

Details

Summary

Experimental branch. Depends on D18739. Ref T2543.

To prepare for --draft, consolidate --preview and --only.

The old --preview is now arc diff --only.

The old --only is now arc diff --only --nounit --nolint.

I think this is more clear than other ways we could do it, since the difference between "draft" and "preview" seems less clear than the difference between "draft" and "only".

Test Plan

Ran arc diff --only and arc diff --only --nounit --nolint.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable