chore: migrate to eslint v9 #150
Conversation
|
Hi @pi0 ! |
eslint.config.mjs
Outdated
| import unjs from "eslint-config-unjs"; | ||
|
|
||
| export default unjs({ | ||
| ignores: ["dist", "node_modules", "src/_parser.ts", "coverage"], |
There was a problem hiding this comment.
I'm pretty sure dist and node_modules are already ignored (otherwise, we should ignore them in the shared config).
coverage should be ignore in the shared config.
Why ignoring src/_parser.ts.
There was a problem hiding this comment.
Hello @Barbapapazes , thank you for your comment.
I checked the settings for dist and coverage in eslint-config-unjs. Therefore, I will delete it.
However, with the current settings, node_modules was not included in this (I don't know why).
src/_parser.ts was defined in the deleted .eslintignore.
I don't know why I added it, so I left it as is.
package.json
Outdated
| @@ -33,12 +33,12 @@ | |||
| "consola": "^3.2.3" | |||
| }, | |||
| "devDependencies": { | |||
| "@types/node": "^20.12.7", | |||
There was a problem hiding this comment.
This change is not related to this PR.
There was a problem hiding this comment.
reverted in this commit: 4c0688c
🔗 Linked issue
closed #144 .
❓ Type of change
📚 Description
migrated ESLint v9 using eslint-config-unjs
v0.3.2📝 Checklist