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!: switch to eslint-plugin-node's maintained fork #660

Merged
merged 3 commits into from
May 27, 2022

Conversation

aladdin-add
Copy link
Contributor

move to eslint-plugin-n: https://www.npmjs.com/package/eslint-plugin-n. as the original one is no longer actively maintained(last version was released ~2y ago).

the fork has been adopted by standard.

@sindresorhus
Copy link
Member

Have you tried contacting the maintainer of eslint-plugin-node?

One issue with switching is that it's going to cause a lot of churn for people ignoring the eslint-plugin-node rules...

@aladdin-add
Copy link
Contributor Author

Yes, I tried, but have not gotten a response from him. 😂

@aladdin-add
Copy link
Contributor Author

@sindresorhus
Copy link
Member

Can you fix the merge conflict?

@aladdin-add
Copy link
Contributor Author

should be addressed now.

package.json Outdated Show resolved Hide resolved
@sindresorhus sindresorhus merged commit 3a4c9c9 into xojs:main May 27, 2022
@aladdin-add aladdin-add deleted the switch-eslint-plugin-n branch May 27, 2022 04:01
kevinoid added a commit to kevinoid/eslint-config-kevinoid that referenced this pull request Jul 3, 2022
Since eslint-plugin-node has been dormant for quite a while:
mysticatea/eslint-plugin-node#294
mysticatea/eslint-plugin-node#300
eslint-plugin-n is an actively maintained fork which is used by xo:
xojs/xo#660
switch to that fork.

Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
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