-
-
Notifications
You must be signed in to change notification settings - Fork 5.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
rework eslint config #11615
rework eslint config #11615
Conversation
b73a3a7
to
09035a3
Compare
I can undo the config/dependency changes if requested. There is a certain benefit of depending on a shared config because much less work is required to update the config, even thought we need to disable 20-something rules that are too opinionated in the airbnb config. |
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.
🎉
- use explicit config that only enables rules - upgrade eslint to 7.1.0 - add new plugins with selected rules enabled - fix discovered issues, remove global wipPrefixes
Updated eslint and plugins again. |
7b06e85
to
baa2932
Compare
Rebased and fixed one more new violation (detected by |
ping LG-TM |
* rework eslint config - use explicit config that only enables rules - upgrade eslint to 7.1.0 - add new plugins with selected rules enabled - fix discovered issues, remove global wipPrefixes * remove if * undo template change * add disabled rules as well for easier config updating * add missing disabled rule * update eslint and plugins * fix new violation * remove deprecated rules Co-authored-by: Lauris BH <lauris@nix.lv>
Rules are mostly based on my config. Two of the plugins still don't specify eslint 7 compatibilty, but I think it's no issue, they do work fine.