Skip to content
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

⭐️ New: Support ESLint 5 #504

Closed
wants to merge 2 commits into from

Conversation

ChristianMurphy
Copy link

@ChristianMurphy ChristianMurphy commented Jun 24, 2018

https://eslint.org/blog/2018/06/eslint-v5.0.0-released
ESLint 5 drops support for Node 4, updated CI accordingly.

@ChristianMurphy ChristianMurphy changed the title Support ESLint 5 ⭐️ New: Support ESLint 5 Jun 24, 2018
@mysticatea
Copy link
Member

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.

@damanwiththeplan
Copy link
Contributor

How about now

@michalsnik
Copy link
Member

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

@michalsnik
Copy link
Member

I'm closing this one in favor of #532 where @mysticatea updated also mentioned parser :)

@michalsnik michalsnik closed this Jul 24, 2018
@ChristianMurphy ChristianMurphy deleted the eslint-5 branch July 24, 2018 13:01
clarkdo added a commit to nuxt/nuxt that referenced this pull request Aug 31, 2018
- [ ] 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants