chore: upgraded eslint to v9 and configure typescript compatible rules#280
chore: upgraded eslint to v9 and configure typescript compatible rules#280
Conversation
WalkthroughReplaces legacy ESLint configuration and ignore file with a Flat Config setup: adds eslint.config.js and removes .eslintrc.js and .eslintignore. Updates ESLint and related devDependencies in package.json and adds the globals package. Bumps Node target in .nvmrc from 16.15.0 to 20.20.0. Updates tsconfig.json to add skipLibCheck and exclude node_modules. Removes several ESLint disable comments in three source files. No functional application logic was introduced. Estimated code review effort🎯 4 (Complex) | ⏱️ ~45 minutes 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 3
🤖 Fix all issues with AI agents
In @.nvmrc:
- Line 1: Update the README.md Node.js version reference from 16.15.0 to match
.nvmrc (20.20.0): find the line in README.md that mentions "Node.js 16.15.0" and
replace it with "Node.js 20.20.0" and optionally add a short note to reference
the .nvmrc file as the authoritative runtime version.
In `@eslint.config.js`:
- Around line 35-37: Remove the undefined "prettier/prettier" rule from the
rules object in eslint.config.js (the entry currently set to "off") because
eslint-plugin-prettier is not installed; either delete that rule line from the
rules block or, if you prefer to keep it, add eslint-plugin-prettier to
devDependencies and register it in the flat config so the "prettier/prettier"
rule is available.
- Line 33: The config currently assigns compat.extends(...) to an extends
property inside a config object (see usage of compat.extends in
eslint.config.js), but FlatCompat returns an array that must be spread into the
top-level config array; fix by removing the extends property from those config
objects and instead spread ...compat.extends(...) directly into the exported
top-level array where other config entries live (apply the same change for both
occurrences of compat.extends at the two locations).
Summary by CodeRabbit