chore(deps-dev): bump eslint from 8.57.1 to 10.0.3#5
chore(deps-dev): bump eslint from 8.57.1 to 10.0.3#5dependabot[bot] wants to merge 1 commit intomainfrom
Conversation
Bumps [eslint](https://github.com/eslint/eslint) from 8.57.1 to 10.0.3. - [Release notes](https://github.com/eslint/eslint/releases) - [Commits](eslint/eslint@v8.57.1...v10.0.3) --- updated-dependencies: - dependency-name: eslint dependency-version: 10.0.3 dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
LabelsThe following labels could not be found: Please fix the above issues or remove invalid values from |
| "@typescript-eslint/parser": "^7.1.0", | ||
| "@vitest/coverage-v8": "^4.0.18", | ||
| "eslint": "^8.57.0", | ||
| "eslint": "^10.0.3", |
There was a problem hiding this comment.
Bug: The upgrade to ESLint v10 is incomplete. It doesn't migrate the configuration, update CLI flags, or upgrade @typescript-eslint packages, which will cause linting to fail.
Severity: CRITICAL
Suggested Fix
To fix this, migrate the .eslintrc.json configuration to the new eslint.config.js flat config format. Update the lint scripts in package.json to remove the deprecated --ext flag. Finally, upgrade the @typescript-eslint/eslint-plugin and @typescript-eslint/parser packages to a version compatible with ESLint v10 (e.g., v8.x or higher).
Prompt for AI Agent
Review the code at the location below. A potential bug has been identified by an AI
agent.
Verify if this is a real issue. If it is, propose a fix; if not, explain why it's not
valid.
Location: package.json#L73
Potential issue: The pull request upgrades ESLint from v8.57.1 to v10.0.3, but this
introduces several breaking changes that are not addressed. ESLint v10 removes support
for `.eslintrc.json` configuration files and the `--ext` CLI flag, both of which are
used in `package.json`. Additionally, the currently used versions of
`@typescript-eslint/eslint-plugin` and `@typescript-eslint/parser` (v7.18.0) have a peer
dependency on ESLint v8 and are incompatible with v10. As a result, running `npm run
lint` will either fail with an error, run with no rules applied, or crash, preventing
the CI check from passing or allowing code with linting errors to be merged.
Did we get this right? 👍 / 👎 to inform future reviews.
|
Superseded by #17. |
Bumps eslint from 8.57.1 to 10.0.3.
Release notes
Sourced from eslint's releases.
... (truncated)
Commits
bfce7ea10.0.3d44ced8Build: changelog update for 10.0.3e511b58fix: update eslint (#20595)ef8fb92chore: package.json update for eslint-config-eslint releasee8f2104chore: updates for v9.39.4 release5cd1604refactor: simplify isCombiningCharacter helper (#20524)9fc31b0docs: Update README70ff1d0chore: eslint-config-eslint require Node^20.19.0 || ^22.13.0 || >=24(#20586)f4c9cf9fix: include variable name inno-useless-assignmentmessage (#20581)4efaa36docs: add info box foreslint-plugin-eslint-comments(#20570)Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)