-
-
Notifications
You must be signed in to change notification settings - Fork 33
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
fix: eslint 7.0 support #156
Conversation
a3890fb
to
e5c6f23
Compare
d6b0a46
to
3a7c5fe
Compare
3a7c5fe
to
e84e65f
Compare
It looks like we may need to bump the lowest eslint version to
|
it could be a breaking to bump the required eslint version. The change is just a bugfix - regardless, the eslint v7 support will be likely dropped in v17. |
What's the reason for dropping eslint v7 support in the next eslint-plugin-n release? |
When we release major releases, we will consider dropping support for some of the older eslint, and will just support the latest 2 major versions (as we have done in the past). Do you have any concerns?🤔 |
I guess I was hoping to get the enhanced-resolve changes without having to migrate to eslint 8, but that's just because I'm trying to be lazy 😃 I also didn't realize there was an eslint 9 alpha release already. |
fixes #155