Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add an option to make tslint results count as error #8

Closed
pgatilov opened this issue Nov 5, 2015 · 3 comments
Closed

Add an option to make tslint results count as error #8

pgatilov opened this issue Nov 5, 2015 · 3 comments

Comments

@pgatilov
Copy link

pgatilov commented Nov 5, 2015

Perhaps this should be fixed by the tslint team (they even have palantir/tslint#345). However, at this point tslint does not distinguish error severity.

I would like Web Analyzer to count any output of tslint as an error. We are starting a new project and this option could enforce discipline. Since not everyone might be this lucky to have no errors, this clearly should be an opt-in.

@richardversluis
Copy link

I would like this too. Reading the intro of this repository it looks like it is supported:

The individual rules for each linter can be configured to be either an error or a warning.

But I haven't figured out how.

@pgatilov
Copy link
Author

@richardversluis I believe that sentence refers to configuration files of corresponding linters. At least ES and CSS linters allow assigning warning/error level to each rule. Don't know about CoffeeScript. So the problem is that TSLint does not have that feature yet.

@richardversluis
Copy link

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants