From 89de113787af46600a6bca218781983dc298e8ad Mon Sep 17 00:00:00 2001 From: Joshua Spence Date: Sat, 10 May 2014 01:52:57 -0700 Subject: [PATCH] Add `--quiet` flags to the `ArcanistCSSLintLinter`. Summary: This isn't really necessary, nor does it bear any significant benefits, however semantically it seems to make sense. From `csslint --help`, the `--quiet` flag causes `csslint` to "Only output when errors are present". Test Plan: Tested using `csslint` directly: ``` > csslint test.css csslint: No errors in /home/joshua/workspace/github.com/facebook/arcanist/foo.css. > csslint --quiet test.css ``` Reviewers: chad, epriestley, #blessed_reviewers Reviewed By: epriestley, #blessed_reviewers Subscribers: epriestley, Korvin Differential Revision: https://secure.phabricator.com/D9036 --- src/lint/linter/ArcanistCSSLintLinter.php | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/lint/linter/ArcanistCSSLintLinter.php b/src/lint/linter/ArcanistCSSLintLinter.php index 25b1afe0..1ac7e4a0 100644 --- a/src/lint/linter/ArcanistCSSLintLinter.php +++ b/src/lint/linter/ArcanistCSSLintLinter.php @@ -19,7 +19,10 @@ final class ArcanistCSSLintLinter extends ArcanistExternalLinter { } public function getMandatoryFlags() { - return array('--format=lint-xml'); + return array( + '--format=lint-xml', + '--quiet', + ); } public function getDefaultFlags() {