eslint-plugin-node icon indicating copy to clipboard operation
eslint-plugin-node copied to clipboard

Support ESLint 8.x

Open MichaelDeBoey opened this issue 4 years ago • 14 comments

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

MichaelDeBoey avatar Aug 23 '21 01:08 MichaelDeBoey

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?

voxpelli avatar Oct 14 '21 20:10 voxpelli

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!

mrtnzlml avatar Oct 26 '21 14:10 mrtnzlml

@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

voxpelli avatar Oct 26 '21 14:10 voxpelli

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

it just added supports for eslint v8.

aladdin-add avatar Dec 17 '21 08:12 aladdin-add

@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 👍

voxpelli avatar Dec 17 '21 09:12 voxpelli

@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

MichaelDeBoey avatar Dec 17 '21 14:12 MichaelDeBoey

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

sosukesuzuki avatar Dec 17 '21 14:12 sosukesuzuki

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

voxpelli avatar Dec 17 '21 15:12 voxpelli

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

MichaelDeBoey avatar Dec 17 '21 15:12 MichaelDeBoey

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

aladdin-add avatar Dec 18 '21 05:12 aladdin-add

Let me know if I can help somehow!

MichaelDeBoey avatar Dec 18 '21 06:12 MichaelDeBoey

shy bump :sweat:

tennox avatar Feb 19 '22 11:02 tennox

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

niksy avatar Mar 20 '22 07:03 niksy

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.

voxpelli avatar Aug 13 '23 12:08 voxpelli