replaced eslint-plugin-node with eslint-plugin-n

[andolf]

eslint-plugin-node seems to no longer be maintained, but a fork has appeared which fixes a lot of issues: eslint-plugin-n. This should enable us to use packages which are now entirely written in ESM.

Bonus: Bumped some deps to latest minor.

[andolf]

This might not actually solve my problem which is being able to install ESM packages (it's still a good change I think).

Main problem is that eslint-plugin-import cannot resolve ESM packages and has no current plans to fix it until resolve can handle it:
import-js/eslint-plugin-import#2331
import-js/eslint-plugin-import#2480
import-js/eslint-plugin-import#1810
browserify/resolve#224

[martindanielson]

Kanske bara ta bort pluginen? Om jag tolkar det rätt så har vi 3 regler från den:

const nodeRecommendedRules = {
  "node/handle-callback-err": "error",
  "node/no-path-concat": "error",
  "node/no-process-exit": "error",
};

Värt att behålla bara för dem? Eller ligger det fler gömda under "recommended" kanske?

[andolf]

Kanske bara ta bort pluginen? Om jag tolkar det rätt så har vi 3 regler från den:

const nodeRecommendedRules = {
  "node/handle-callback-err": "error",
  "node/no-path-concat": "error",
  "node/no-process-exit": "error",
};

Värt att behålla bara för dem? Eller ligger det fler gömda under "recommended" kanske?

Rent generellt tycker jag de är bra men det finns också en annan regel som jag vill ha in i framtiden: no-deprecated-api

Vi verkar inte använda recommended-setet men det finns massor av bra grejer där med som kan vara värt att kika på efteråt.
Komplett lista här: https://github.com/weiran-zsd/eslint-plugin-node#-rules

[martindanielson]

Kanske bara ta bort pluginen? Om jag tolkar det rätt så har vi 3 regler från den:

const nodeRecommendedRules = {
  "node/handle-callback-err": "error",
  "node/no-path-concat": "error",
  "node/no-process-exit": "error",
};

Värt att behålla bara för dem? Eller ligger det fler gömda under "recommended" kanske?

Rent generellt tycker jag de är bra men det finns också en annan regel som jag vill ha in i framtiden: no-deprecated-api

Vi verkar inte använda recommended-setet men det finns massor av bra grejer där med som kan vara värt att kika på efteråt. Komplett lista här: https://github.com/weiran-zsd/eslint-plugin-node#-rules

Ok, men jag godkänner denna. Lite läskigt bara med saker som inte underhålls helt. Kikade runt lite vad andra gör men det verkar som att folk istället för att använda andra libar kör helt andra lösningar är lint, snabbare alternativ med liknande men oftast en mer komplett lösning för kodvalidering etc. Nått värt att gräva i men inte just nu kanske :)

[martindanielson]

🌟

[andolf]

Ok, men jag godkänner denna. Lite läskigt bara med saker som inte underhålls helt.

Fast nu förstår jag inte riktigt... Jag byter från eslint-plugin-node som inte underhålls längre till eslint-plugin-n som har bättre support och fortfarande underhålls :)