HomePhabricator

Don't pass `--quiet` to `coffeelint`

Description

Don't pass --quiet to coffeelint

Summary: This flag doesn't do anything when combined with --reporter=checkstyle.

Test Plan:

> coffeelint *.coffee
  ✗ fail.coffee
     ✗ #1: Class names should be camel cased. class name: boaConstrictor.
  ✓ pass.coffee

✗ Lint! » 1 error and 0 warnings in 2 files

> coffeelint --quiet *.coffee
  ✗ fail.coffee
     ✗ #1: Class names should be camel cased. class name: boaConstrictor.

✗ Lint! » 1 error and 0 warnings in 2 files

> coffeelint --reporter=checkstyle *.coffee
<?xml version="1.0" encoding="utf-8"?>
<checkstyle version="4.3">
<file name="fail.coffee">
<error line="1"
    severity="error"
    message="Class names should be camel cased; context: class name: boaConstrictor"
    source="coffeelint"/>
</file>
</checkstyle>

> coffeelint --reporter=checkstyle --quiet *.coffee
<?xml version="1.0" encoding="utf-8"?>
<checkstyle version="4.3">
<file name="fail.coffee">
<error line="1"
    severity="error"
    message="Class names should be camel cased; context: class name: boaConstrictor"
    source="coffeelint"/>
</file>
</checkstyle>

Reviewers: epriestley, Blessed Reviewers

Reviewed By: epriestley, Blessed Reviewers

Subscribers: Korvin, epriestley

Differential Revision: https://secure.phabricator.com/D11198

Details

Provenance
joshuaspenceAuthored on
joshuaspencePushed on Jan 4 2015, 7:49 PM
Reviewer
Blessed Reviewers
Differential Revision
D11198: Don't pass `--quiet` to `coffeelint`
Parents
rARCda02add6c818: Create an `ArcanistExternalLinterTestCase` class
Branches
Unknown
Tags
Unknown