-
Notifications
You must be signed in to change notification settings - Fork 49
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 auto-fix of misspelled falsey
to falsy
#229
Conversation
Hi, It feels like this should be in a separate rule as this is the only error "fixable". The error message could be better, but I've gone with simplicity.
|
Thanks for all your PRs @GMartigny!
|
You need to update the readme and rule docs to indicate it's fixable. In the rule docs, you can clarify exactly what is fixable. |
I think it's fine to keep it in this rule. |
falsey
to falsy
eslint-plugin-ava/test/use-t-well.js Lines 115 to 116 in ab64fde
|
I updated the readme.md, but other fixable rules (no-only-test, no-skip-test) doc don't mention they are fixable. |
That's just an oversight. Would you be willing to fix those in a separate PR? |
Explain what's fixable in the rule docs. Example: https://github.com/sindresorhus/eslint-plugin-unicorn/blob/master/docs/rules/prevent-abbreviations.md#prevent-abbreviations |
@GMartigny Can you add this to the rule documentation for this rule so this PR can be merged? |
fix #190