From 10bdc77f96b04763bb7e02b7d1c3b6268b88b9b9 Mon Sep 17 00:00:00 2001 From: Adam Simpkins Date: Tue, 17 May 2011 12:54:38 -0700 Subject: [PATCH] arc diff: --only doesn't conflict with --nounit or --nolint Summary: The --only flag implies --nounit and --nolint, so there's really no conflict if these flags are specified together. (My wrapper around arc diff automatically specifies --nounit in some cases. It is useful to be able to manually add --only in some cases, and not have it fail because it conflicts with the automatically specified --nounit flag.) Test Plan: Ran "arc diff --only --nounit". Reviewed By: epriestley Reviewers: epriestley, jungejason CC: aran, epriestley, simpkins Differential Revision: 298 --- src/workflow/diff/ArcanistDiffWorkflow.php | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/workflow/diff/ArcanistDiffWorkflow.php b/src/workflow/diff/ArcanistDiffWorkflow.php index 64295f3c..1f5f3f05 100644 --- a/src/workflow/diff/ArcanistDiffWorkflow.php +++ b/src/workflow/diff/ArcanistDiffWorkflow.php @@ -118,8 +118,6 @@ EOTEXT 'advice' => '--only suppresses lint.', 'apply-patches' => '--only suppresses lint.', 'never-apply-patches' => '--only suppresses lint.', - 'nounit' => '--only implies --nounit.', - 'nolint' => '--only implies --nolint.', ), ), 'preview' => array(