-
-
Notifications
You must be signed in to change notification settings - Fork 695
⭐️ New: Support ESLint 5 #504
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
Conversation
63ff601 to
f0cdc8b
Compare
f0cdc8b to
3c3a683
Compare
Thank you for this PR.
I'm seeing a parser problem: vuejs/vue-eslint-parser#26
I think that we have to resolve that issue before support 5.0.0.
How about now
I think this change should be released in a new major version of eslint-plugin-vue. Currently build fails, as vue-eslint-parser requires eslint@5 as peer dependency, and we can't drop Node 4 support out of the blue. I'm going to prepare new beta soon, and will try include this change and drop support for Node 4 there. Thanks for working on it @ChristianMurphy
I'm closing this one in favor of #532 where @mysticatea updated also mentioned parser :)
- [ ] babel-eslint babel/babel-eslint#664 - [x] eslint-config-standard-jsx standard/eslint-config-standard-jsx#32 - [x] eslint-config-standard to be stable release standard/eslint-config-standard#123 - [x] eslint-plugin-html - [x] eslint-plugin-import - [x] eslint-plugin-jest - [x] eslint-plugin-node - [x] eslint-plugin-promise - [x] eslint-plugin-standard standard/eslint-plugin-standard#29 - [x] eslint-plugin-vue vuejs/eslint-plugin-vue#504 - [x] eslint-plugin-react https://github.com/yannickcr/eslint-plugin-react/releases/tag/v7.10.0
Uh oh!
There was an error while loading. Please reload this page.
https://eslint.org/blog/2018/06/eslint-v5.0.0-released
ESLint 5 drops support for Node 4, updated CI accordingly.