Skip to content

chore: Pin eslint to 9.39.3 (eslint@10 incompatible with eslint-plugin-github)#194

Merged
abdulahmad307 merged 2 commits intodependabot/npm_and_yarn/eslint-10.0.2from
copilot/sub-pr-193
Mar 5, 2026
Merged

chore: Pin eslint to 9.39.3 (eslint@10 incompatible with eslint-plugin-github)#194
abdulahmad307 merged 2 commits intodependabot/npm_and_yarn/eslint-10.0.2from
copilot/sub-pr-193

Conversation

Copy link
Contributor

Copilot AI commented Mar 4, 2026

ESLint 10 cannot be used with eslint-plugin-github@6.0.0 for two reasons:

  • Peer dep conflict: eslint-plugin-github@6.0.0 declares peerDependencies: { eslint: "^8 || ^9" } — no released version supports eslint@10
  • Runtime breakage: ESLint 10 removed context.getSourceCode(), which eslint-plugin-eslint-comments (transitive dep via eslint-plugin-github) still calls

Changes

  • Reverts eslint from ^10.0.2^9.39.3 (latest 9.x) in package.json
  • Updates package-lock.json accordingly

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

…ithub

Co-authored-by: abdulahmad307 <204748719+abdulahmad307@users.noreply.github.com>
Copilot AI changed the title [WIP] Bump eslint from 9.39.2 to 10.0.2 chore: Pin eslint to 9.39.3 (eslint@10 incompatible with eslint-plugin-github) Mar 4, 2026
@abdulahmad307 abdulahmad307 marked this pull request as ready for review March 5, 2026 17:11
@abdulahmad307 abdulahmad307 requested a review from a team as a code owner March 5, 2026 17:11
@abdulahmad307 abdulahmad307 merged commit a1b2fe1 into dependabot/npm_and_yarn/eslint-10.0.2 Mar 5, 2026
2 of 3 checks passed
@abdulahmad307 abdulahmad307 deleted the copilot/sub-pr-193 branch March 5, 2026 17:11
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Pins ESLint to a compatible 9.x version to avoid peer dependency conflicts and runtime breakage with eslint-plugin-github@6.0.0.

Changes:

  • Downgrades ESLint from ^10.0.2 to ^9.39.3 in package.json
  • Updates the lockfile to reflect the ESLint downgrade

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

},
"devDependencies": {
"eslint": "^10.0.2",
"eslint": "^9.39.3",
Copy link

Copilot AI Mar 5, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR title says "Pin eslint to 9.39.3", but ^9.39.3 is not a pin—it allows automatic upgrades up to (but not including) 10.0.0. If the intent is a true pin to avoid future incompatibilities, change this to "eslint": "9.39.3" (or use ~9.39.3 if you only want patch updates).

Suggested change
"eslint": "^9.39.3",
"eslint": "9.39.3",

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants