-
Notifications
You must be signed in to change notification settings - Fork 2k
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
eslint: upgrade to 1.10.3 #1807
Conversation
a3bc840
to
43f5d3b
Compare
Fix undef error in ad-tracking
Works for me! Should we fix the lint-reported errors in this PR also? eg:
|
Yes, have to, as they break the build. I was just pushing this up to track the breakage. Should have marked it |
I fixed a pile of them. The few remaining I'm not sure about (not sure how to use |
ping @codebykat for the tinymce errors:
|
Yep. It's attached to the |
Ping @deBhal for the translator
|
Fixed up the editor charmap in c415e0f. TinyMCE is a global but we shouldn't use it like that unless absolutely necessary. |
The notices fix looked simple enough too ( I think this is good to go 👍 |
Move to 1.10.3 and 3.11.3 of the react plugin
To test, run
make lint
and it doesn't break