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 fixable property to fixable rules (and mention in docs) #228

Merged
merged 1 commit into from
Feb 17, 2020

Conversation

brettz9
Copy link
Contributor

@brettz9 brettz9 commented Feb 17, 2020

Per docs at https://eslint.org/docs/developer-guide/working-with-rules :
"Without the fixable property, ESLint does not apply fixes even if the rule implements fix functions. Omit the fixable property if the rule is not fixable."

Also fixes one non-alphetizated rule.

I've also switched to the non-deprecated. I could also add for the other rules as well, but in this PR I've only changed those where it was a natural change as a part of adding the fixable property.

Per docs at https://eslint.org/docs/developer-guide/working-with-rules :
  "Without the fixable property, ESLint does not apply fixes even if the rule implements fix functions. Omit the fixable property if the rule is not fixable."
Copy link
Owner

@lo1tuma lo1tuma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, I greatly appreciate it.

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

Successfully merging this pull request may close these issues.

None yet

2 participants