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

Warning for style #13

Closed
denis-sokolov opened this issue Sep 11, 2015 · 3 comments
Closed

Warning for style #13

denis-sokolov opened this issue Sep 11, 2015 · 3 comments

Comments

@denis-sokolov
Copy link

Is there a design decision on why the warnings feature of eslint is not used? One could consider marking only the real error-catching rules as errors, and the rest of styling distinctions as warnings.

This allows the users to prioritize efforts on fixing errors first, and the style second. More importantly, the currently proposed best practice of running standard as part of npm test results in quick test cycle (i.e. TDD) users being highly annoyed, as it is impossible to iterate quickly if every tiny styling issue breaks the tests. Warnings would not affect the return code, resulting in a better experience.

@denis-sokolov
Copy link
Author

Disregard that, did not find standard/standard#108 before, sorry.

@feross
Copy link
Member

feross commented Sep 11, 2015

No problem. Thanks for opening a thoughtful issue. :)

@marcinn
Copy link

marcinn commented Feb 18, 2021

I read the docs and this discussion. Just want to say that "All rules should be errors" is an insane rule. This makes "the standard/standard" unusable on the battlefield. You want to shoot an enemy, but your gun says "error, wrong finger". Nah.

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

No branches or pull requests

3 participants