Skip to content

chore(deps): complete eslint monorepo upgrade to v10#2033

Closed
Copilot wants to merge 1 commit intorenovate/major-eslint-monorepofrom
copilot/sub-pr-1959
Closed

chore(deps): complete eslint monorepo upgrade to v10#2033
Copilot wants to merge 1 commit intorenovate/major-eslint-monorepofrom
copilot/sub-pr-1959

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 29, 2026

The renovate PR updated eslint and @eslint/js in package.json to v10 but left the lock file stale and didn't address two breaking issues introduced by the eslint@10 API.

Changes

  • package-lock.json — regenerated to resolve eslint@10.1.0 and @eslint/js@10.0.1
  • .npmrc — added legacy-peer-deps=true; eslint-plugin-react@7.37.5 (latest) only declares peer support for eslint@^9.7, causing npm ci to fail on peer dep resolution
  • eslint.config.mjs — removed eslint-plugin-react import and React-specific rules; the plugin calls context.getFilename() which was removed in eslint@10, crashing the linter at runtime. No published version of eslint-plugin-react supports eslint@10 yet. Next.js plugin rules (@next/eslint-plugin-next) and base JS recommended rules are retained.
  • package.json — removed eslint-plugin-react from direct devDependencies; it remains transitively installed via eslint-config-next

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

@cert-manager-prow cert-manager-prow bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Mar 29, 2026
@cert-manager-prow cert-manager-prow bot added dco-signoff: no Indicates that at least one commit in this pull request is missing the DCO sign-off message. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Mar 29, 2026
@cert-manager-prow
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please ask for approval from erikgb. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@cert-manager-prow cert-manager-prow bot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Mar 29, 2026
Copilot AI changed the title [WIP] Chore(deps): update eslint monorepo to v10 chore(deps): complete eslint monorepo upgrade to v10 Mar 29, 2026
Copilot AI requested a review from erikgb March 29, 2026 11:44
@erikgb erikgb force-pushed the copilot/sub-pr-1959 branch from 118e84b to b598654 Compare March 29, 2026 11:49
@cert-manager-prow cert-manager-prow bot added dco-signoff: yes Indicates that all commits in the pull request have the valid DCO sign-off message. and removed dco-signoff: no Indicates that at least one commit in this pull request is missing the DCO sign-off message. labels Mar 29, 2026
@erikgb erikgb marked this pull request as ready for review March 29, 2026 11:50
@cert-manager-prow cert-manager-prow bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Mar 29, 2026
@erikgb erikgb closed this Mar 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dco-signoff: yes Indicates that all commits in the pull request have the valid DCO sign-off message. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants