-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
use semistandard #2490
Comments
Great move! 👍 |
Merged
boneskull
removed
the
status: accepting prs
Mocha can use your help with this one!
label
Oct 10, 2016
This was referenced Sep 23, 2018
1 task
1 task
This was referenced Dec 23, 2018
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We have some code standards in place, but not enough. We have disabled many warnings which we probably shouldn't have, to avoid changing too much code when we picked up ESLint originally. I'd also prefer to use a widely-used style instead of a custom style.
.eslintrc
to be just{"extends": "semistandard"}
eslint --fix
everythingIf anyone disagrees, please let me know. Unless that disagreement involves "semicolons"; if so, please keep it to yourself. 😄
The text was updated successfully, but these errors were encountered: