mysticatea / eslint-plugin-node

Additional ESLint's rules for Node.js
MIT License
958 stars 167 forks source link

Support ESLint 8.x #294

Open MichaelDeBoey opened 2 years ago

MichaelDeBoey commented 2 years ago

ESLint v8.0.0 is released 🎉

It would be awesome to have official ESLint 8 support. 👊 I'm happy to help where I can of course 🙂


Follow-up of #223

voxpelli commented 2 years ago

Is anything broken in the current release and needs fixing for ESLint 8?

This module automatically supports ESLint 8: https://github.com/mysticatea/eslint-plugin-node/blob/f45c6149be7235c0f7422d1179c25726afeecd83/package.json#L12-L14

So unless something is broken, then this is a non-issue?

mrtnzlml commented 2 years ago

Is anything broken in the current release and needs fixing for ESLint 8?

Hi @voxpelli! I have one thing that is broken. The following file is currently not behaving correctly with Eslint version 8: https://github.com/adeira/universe/blob/8cff40fc01c2aca6ae01ec53cd8d1be07afd8c84/src/eslint-config-adeira/__tests__/__fixtures__/valid-eslint-examples/node/process-exit-as-throw.js

We created this file based on the official example because of some previous problems with this rule. Here is the error we are getting:

Screen Shot 2021-10-26 at 9 06 31

Thanks for having a look!

voxpelli commented 2 years ago

@mrtnzlml Great example! Can you add a dedicated issue for it, like a ESLint 8.x regression: node/process-exit-as-throw no longer works

aladdin-add commented 2 years ago

for someone blocked by the plugin, I've just pushed a fork: eslint-plugin-n.

it just added supports for eslint v8.

voxpelli commented 2 years ago

@aladdin-add Can you summarize somewhere on how it relates to the ESLint 8 PR:s in this repo? What the changes are from upstream? + maybe links to PR:s for those changes?

Great to have a fork to use until upstream gets updated 👍

MichaelDeBoey commented 2 years ago

@aladdin-add Would be nice if we could transfer repos like eslint-plugin-node, eslint-plugin-es, eslint-utils & @mysticatea/eslint-plugin to a new @eslint-community org

CC/ @mysticatea

sosukesuzuki commented 2 years ago

Thank you. I agree with @MichaelDeBoey. It would be nice to be able to take over maintenance and release from @mysticatea

voxpelli commented 2 years ago

As we need them for StandardJS as well, I think we can maybe add them there if we want a common org as well

MichaelDeBoey commented 2 years ago

Transferring them to the @standard org is fine for me too 🤷‍♂️ I just think we should transfer it to an org that will take care of maintenance long term, that's all

Just need to get in touch with @mysticatea somehow to get it done and then I think a lot of PRs can get merged quickly

aladdin-add commented 2 years ago

nicholas also agreed to the eslint-community, will put together an RFC for how it will work.

MichaelDeBoey commented 2 years ago

Let me know if I can help somehow!

tennox commented 2 years ago

shy bump :sweat:

niksy commented 2 years ago

It seems like eslint-plugin-n is maintained version of this plugin.

voxpelli commented 11 months ago

eslint-plugin-n is the maintained version of this module.

We switched to it in eg. eslint-config-standard / standard and it is maintained by me and other members of the official ESLint community organization.