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

feat: move to eslint-plugin-n #765

Closed

Conversation

aladdin-add
Copy link

eslint-plugin-n is a community-maintained fork of eslint-plugin-node: https://github.com/eslint-community/eslint-plugin-n

It ensures that an important module like eslint-plugin-node is kept up to date, with both ESLint 8.x and with Node.js itself.

Similar PR: electron/electron#38874

Full disclosure: I'm part of the ESLint Community Core Team.

@aladdin-add aladdin-add changed the title chore: move to eslint-plugin-n feat: move to eslint-plugin-n Jun 30, 2023
@aladdin-add
Copy link
Author

aladdin-add commented Jun 30, 2023

looks like the package still supports node.js v12, while eslint-plugin-n has dropped node.js < 16 (EOL). You can merge the PR after dropping node.js < 16. :)

refs: https://github.com/nodejs/release#end-of-life-releases


no worries, I've reverted it to eslint-plugin-n v15.

eslint-plugin-n is a community maintained fork of eslint-plugin-node:
https://github.com/eslint-community/eslint-plugin-n

It ensures that an important module like eslint-plugin-node is kept up
to date, with both ESLint 8.x and with Node.js itself.

Full disclosure: I'm part of the ESLint Community Core Team.
@tonycoco
Copy link
Contributor

Going to track changes here: #865 with tests fixed and passing

@tonycoco tonycoco closed this Mar 22, 2024
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.

2 participants