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
Implement autofix for no-implicit-coercion #7056
Comments
Just for the convenience of anyone perusing the issue, could you show the messages you receive so we can see exactly what fixes you have in mind? |
Yeah, I plan on digging into this. I updated the current tests with example fixes: https://github.com/TheSavior/eslint/blob/fix-conversion/tests/lib/rules/no-implicit-coercion.js#L80 |
Essentially, this rule right now reports exactly what to replace each node with. I recommend that when using the fix option, it just does that exact change. |
👍 from me. Given that we already calculate the replacement for the error message, should be pretty straight forward! |
👍 from me as well |
👍 Just need somebody to champion this now. |
I'll champion. |
What version of ESLint are you using?
3.0.0
What parser (default, Babel-ESLint, etc.) are you using?
default
Please show your full configuration:
Seems unnecessary
What did you do? Please include the actual source code causing the issue.
I tried to use the --fix command with
What did you expect to happen?
I expected it to fix my source
What actually happened? Please include the actual, raw output from ESLint.
It gave me explicit recommendations on what to write in my code, but didn't make the actual changes.
The text was updated successfully, but these errors were encountered: