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
[core] Remove unnecessary rules in .eslintrc #14173
[core] Remove unnecessary rules in .eslintrc #14173
Conversation
@WebDeg-Brian It doesn't work. However @joshwooding has raised an important concern for Windows users We need to add this rule back 35fd3b8#diff-e4403a877d80de653400d88d85e4801aL30. I was too greedy. |
@oliviertassinari ah yes that killer rule. Is there a reason that I can no longer update the file? I removed those and everything works fine on my end. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If this would just "remove unnecessary rules" then I don't understand the big diff. There are things changed here that are not included in the description. Let's discuss individual points without making some sweeping changes.
@eps1lon I might as well close and investigate this a bit further. I didn't understand why as soon as I committed the changes the unit tests failed. |
@WebDeg-Brian I'm rebasing. |
22cbf71
to
3c2b6d9
Compare
I have found a fix. |
@oliviertassinari Awesome! |
@oliviertassinari Thank you! |
This reverts commit 733c694.
Remove rules that have been turned off by
eslint-config-prettier