-
-
Notifications
You must be signed in to change notification settings - Fork 638
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
Support eslint v4 #266
Comments
Doesn't seem like anything needs to be done on our end, except upgrade peerDependencies. notice anything in the migration guide/changelog that would affect this plugin? |
@evcohen we should make sure that our peer dep is Generally the changes are "invalid configs throw now", and some rules got stricter - so I don't think it would impact us. |
@ljharb we can use |
That's dangerous, because then a v5 breaking change might break us. Just like you should never do |
No description provided.
The text was updated successfully, but these errors were encountered: