-
Notifications
You must be signed in to change notification settings - Fork 712
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
Update eslint dependencies #2896
Conversation
8f4430f
to
ce97b00
Compare
Is this in sync with the service-ui eslint file? If so, LGTM. |
It's not in sync as the p.s. There was a failing test I had to fix in the meantime but that was a one liner in a test file. |
Resolves #2895.
Everything should feel and work exactly as before, most of the new rules are purely syntactic. Two new rules that weren't applied have to do with converting some
div
s,span
s anda
s intobutton
s and their scope of (semantic) changes is too big for this PR so they're left temporarily disabled.New/updated rules
Disabled rules