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

Please, enable eslint@8 support #208

Closed
zloirock opened this issue Oct 9, 2021 · 4 comments · Fixed by #209
Closed

Please, enable eslint@8 support #208

zloirock opened this issue Oct 9, 2021 · 4 comments · Fixed by #209

Comments

@zloirock
Copy link

zloirock commented Oct 9, 2021

A stable version of eslint@8 is already published.

Now, even on master:

  "peerDepencencies": {
    "eslint": ">=7.0.0 <8.0.0"
  },

So it's a block for eslint@8 usage.

@platinumazure
Copy link
Owner

Thanks for the issue.

eslint@8 was only released yesterday. I had every intention of consuming it as soon as it became available. It's the last item in the major release of this plugin. However, I do have a life and deserve some time to myself. So, I appreciate everyone's patience on this.

@platinumazure
Copy link
Owner

[email protected] contains a number of breaking changes for plugin developers. This package is not impacted directly. However, other ESLint plugins that we consume as devDependencies do not currently work with [email protected], and we can't run the test suite until those plugins are compatible.

Currently, we are blocked until the following PR(s) are merged and released:

@bmish
Copy link
Collaborator

bmish commented Oct 10, 2021

I have a PR open to fix this #209.

@platinumazure
Copy link
Owner

I have a branch locally with devDependency upgrades, which I will need to rebase on top of this fix. Once that's in, I'll release 7.0.0. Goal is to get that done in the next day or two.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants