-
-
Notifications
You must be signed in to change notification settings - Fork 69
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
feat: support ESLint v8 #759
Conversation
9dfdeab
to
f39e0d6
Compare
Possible to publish a pre-release version of this for testing? (because the config file is typescript, can't just use the branch directly in eslint config) |
@jefbarn Yes, we can certainly do a prerelease before a stable release. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm going to leave it to @mightyiam to decide on this one, as he is the one driving this part of standard
Any updates on this? We are waiting for this to be released so we can upgrade to eslint 8. |
@flippidippi It is blocked by: standard/eslint-config-standard#208 |
For people wanting to start using this, my solution was:
Though this solution is not great if we need to merge in changes from upstream. |
1816573
to
dc25743
Compare
@mightyiam This should now be ready! 😄 |
I'll do my best. |
Any updates on the release? eslint-config-standard is released |
fix: log #759 as breaking change
What is the purpose of this pull request? (put an "X" next to item)
[ ] Documentation update
[ ] Bug fix
[x] New feature
[ ] Other, please explain:
What changes did you make? (Give an overview)
Support ESLint v8.
Which issue (if any) does this pull request address?
Fixes #728
Is there anything you'd like reviewers to focus on?
Even if #727, @mightyiam, do you still want to review or we can merge/release with another standard maintainer approval?