kratiahuja / broccoli-tslinter

Broccoli Plugin to run linter on typescript files. Uses tslint internally.
MIT License
9 stars 6 forks source link

Option for "error only" logging output #31

Open mike-north opened 6 years ago

mike-north commented 6 years ago

When working in a project with lots of in-repo addons/engines, the output from this broccoli plugin gets a little noisy. In my example app, rougly 50% of the logging output for the whole app comes from this one broccoli plugin

This kind of output may be best implemented with npm:debug

Using tslint.json as the default file for linting rules
Using tslint.json as the default file for linting rules
Using tslint.json as the default file for linting rules
Using tslint.json as the default file for linting rules
Using tslint.json as the default file for linting rules
Using tslint.json as the default file for linting rules
Using tslint.json as the default file for linting rules
Using tslint.json as the default file for linting rules
Using tslint.json as the default file for linting rules
Using tslint.json as the default file for linting rules
Using tslint.json as the default file for linting rules
Using tslint.json as the default file for linting rules
Using tslint.json as the default file for linting rules
Using tslint.json as the default file for linting rules
Using tslint.json as the default file for linting rules
Using tslint.json as the default file for linting rules
Finished linting 16 successfully
Finished linting 8 successfully
Finished linting 8 successfully
Finished linting 32 successfully
Finished linting 31 successfully
Finished linting 1 successfully
Finished linting 1 successfully
Finished linting 3 successfully
Finished linting 10 successfully
Finished linting 10 successfully
Finished linting 10 successfully
Finished linting 20 successfully
Finished linting 2 successfully
Finished linting 1 successfully
ghost commented 6 years ago

Any known workaround for this?

mike-north commented 6 years ago

@lsg-richard looks like explicitly specifying a configuration will suppress the first message

https://github.com/kratiahuja/broccoli-tslinter/blob/64cc59f5d4dda6b0c1db90da1094c61b96f1a861/index.js#L25-L29

ghost commented 6 years ago

Cool, doesn't look it's currently possible to set this when invoking via ember-cli-tslint though:

https://github.com/typed-ember/ember-cli-tslint/blob/ceaba5d8dc9c2727a8b56ef2355c5c6a08aee858/index.js#L16