From f6b3f3c46a041f1f318187fdfe8dce2e3f9b9b3e Mon Sep 17 00:00:00 2001 From: Joshua Spence Date: Mon, 5 Jan 2015 06:40:07 +1100 Subject: [PATCH] Ignore environment variables for `coffeelint` Summary: According to `coffeelint --help`, `--noconfig` ignores the environment variable `COFFEELINT_CONFIG`. This means that `arc lint` will behave more consistently across developer workstations by forcing configuration to be specified with `coffeelint.config` instead of with environment variables. Test Plan: This is mostly theoretical. Reviewers: #blessed_reviewers, epriestley Reviewed By: #blessed_reviewers, epriestley Subscribers: Korvin, epriestley Differential Revision: https://secure.phabricator.com/D11196 --- src/lint/linter/ArcanistCoffeeLintLinter.php | 1 + 1 file changed, 1 insertion(+) diff --git a/src/lint/linter/ArcanistCoffeeLintLinter.php b/src/lint/linter/ArcanistCoffeeLintLinter.php index ffc73b91..06853770 100644 --- a/src/lint/linter/ArcanistCoffeeLintLinter.php +++ b/src/lint/linter/ArcanistCoffeeLintLinter.php @@ -61,6 +61,7 @@ final class ArcanistCoffeeLintLinter extends ArcanistExternalLinter { $options = array( '--reporter=checkstyle', '--color=never', + '--noconfig', '--quiet', );