deps(deps-dev): bump eslint from 9.39.5 to 10.8.1 - #3206
Conversation
|
You have reached your Codex usage limits for security reviews. Please try again later. |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9ea869c04e
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| "autoprefixer": "^10.5.3", | ||
| "axe-core": "4.12.1", | ||
| "eslint": "9.39.5", | ||
| "eslint": "10.8.1", |
There was a problem hiding this comment.
Keep the UI on a peer-valid ESLint version
With eslint-config-next@16.2.12, this installs ESLint 10 alongside eslint-plugin-import, eslint-plugin-jsx-a11y, and eslint-plugin-react, whose committed peer ranges still end at ESLint 9 (package-lock.json lines 14330, 14370, and 14403), so npm ls eslint reports an invalid dependency tree. The repository's migration evidence in docs/next16-eslint10-ui-migration.md:10-13 explicitly rejected ESLint 10 for this configuration; retain 9.39.5 in both workspaces until those plugins support v10, or upgrade the complete compatible plugin set together.
Useful? React with 👍 / 👎.
9ea869c to
db5df2d
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: db5df2d92a
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| "acorn": "^8.16.0", | ||
| "dependency-cruiser": "^18.0.0", | ||
| "eslint": "^9.39.5", | ||
| "eslint": "^10.8.1", |
There was a problem hiding this comment.
Align the recommended preset with ESLint 10
If ESLint 10 is retained, also upgrade the root @eslint/js, because eslint.config.js:2 imports js.configs.recommended while package.json:233 leaves that preset on 9.39.5. The old preset does not enable ESLint 10's newly recommended no-unassigned-vars, no-useless-assignment, and preserve-caught-error rules, so those defects silently pass the repository's machine-enforced lint gate; keep both packages on 9 or upgrade them together.
AGENTS.md reference: AGENTS.md:L270-L280
Useful? React with 👍 / 👎.
Bumps [eslint](https://github.com/eslint/eslint) from 9.39.5 to 10.8.1. - [Release notes](https://github.com/eslint/eslint/releases) - [Commits](eslint/eslint@v9.39.5...v10.8.1) --- updated-dependencies: - dependency-name: eslint dependency-version: 10.8.1 dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
db5df2d to
edf38b2
Compare
|
Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits. |
Bumps eslint from 9.39.5 to 10.8.1.
Release notes
Sourced from eslint's releases.
... (truncated)
Commits
c049dc310.8.1a3f7826Build: changelog update for 10.8.118eb0a7fix: prevent ASI hazard inno-unused-labelsautofix (#21173)0a14800chore: update github/codeql-action action to v4.37.4 (#21196)7d0cbf8docs: Update README05adcb1test: fix failing ecosystem test foreslint-plugin-unicorn(#21191)5611035test: add error locations info tono-void(#21185)ee47333ci: bump github/codeql-action from 4 to 4.37.3 (#21176)f131c03chore: improve ecosystem test failure reporting (#20937)0a05812docs: add missing backticks tono-duplicate-imports.js(#21183)