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

breaking: drop eslint < 7 & node.js < 10 #95

Merged
merged 1 commit into from
Apr 8, 2021

Conversation

MichaelDeBoey
Copy link
Member

@MichaelDeBoey MichaelDeBoey commented May 8, 2020

ESLint v7.0.0 is released 🎉

devDependencies should be compatible with ESLint 7 too before we can merge this one:


BREAKING CHANGE: Requires Node@^10.12.x || 12.x
BREAKING CHANGE: Requires ESLint@^7.x

Closes #94

BREAKING CHANGE: Requires Node@^10.12.x || 12.x
BREAKING CHANGE: Requires ESLint@^7.x
@MichaelDeBoey MichaelDeBoey marked this pull request as ready for review September 20, 2020 13:29
@MichaelDeBoey
Copy link
Member Author

@not-an-aardvark Since all tests are green for ESLint 7.x, I think we can merge this one without mysticatea/eslint-plugin-node#224 being merged 🙂

Copy link
Contributor

@aladdin-add aladdin-add left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@MichaelDeBoey
Copy link
Member Author

@aladdin-add Any idea when this can be merged and released?

@xjamundx
Copy link

xjamundx commented Apr 7, 2021 via email

@aladdin-add
Copy link
Contributor

will have to wait @not-an-aardvark to do a new release, I don't have the access to publish to npm:)

it was written "eslint": ">=5.0.0", so the latest eslint-plugin-eslint-plugin has already supported the latest eslint(7.x). is it blocking something else?

@aladdin-add aladdin-add changed the title feat: Support ESLint 7.x breaking: drop eslint < 7 & node.js < 10 Apr 7, 2021
@aladdin-add
Copy link
Contributor

I've reedited the title, as it is a breaking change. :)

@not-an-aardvark
Copy link
Contributor

@aladdin-add I just gave you publish access on npm, feel free to publish when you think it's appropriate.

@aladdin-add aladdin-add merged commit 59ddffc into eslint-community:master Apr 8, 2021
@aladdin-add
Copy link
Contributor

released in https://www.npmjs.com/package/eslint-plugin-eslint-plugin/v/3.0.0

@MichaelDeBoey MichaelDeBoey deleted the eslint-7 branch April 8, 2021 10:39
@ljharb
Copy link

ljharb commented Jul 28, 2021

Dropping eslint < 7 is very unfortunate, since it means eslint plugins (which often depend on multiple versions of eslint, including the previous major) can't use it without getting npm7 peer deps.

eslint-plugin-react is stuck on v2 of this plugin for effectively ever, unless eslint 6 compat, at least, can be restored.

@aladdin-add
Copy link
Contributor

a workaround is to use the --legacy-peer-deps.

oc, it seems no harm to restore the eslint v6 compat. will try to release a new version later.👍

@ljharb
Copy link

ljharb commented Jul 29, 2021

Yes, that is indeed what I'm using :-) and thank you, that would be most helpful.

aladdin-add added a commit that referenced this pull request Jul 29, 2021
aladdin-add added a commit that referenced this pull request Jul 29, 2021
* Fix: support eslint v6

refs: #95 (comment)

* chore: simplify eslint config
@aladdin-add
Copy link
Contributor

released in v3.5.2. 😄

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 this pull request may close these issues.

Support ESLint 7.x
5 participants