Skip to content

fix: downgrade to ESLint 9 temporarily#284

Merged
JounQin merged 1 commit intomainfrom
fix/eslint
Mar 12, 2026
Merged

fix: downgrade to ESLint 9 temporarily#284
JounQin merged 1 commit intomainfrom
fix/eslint

Conversation

@JounQin
Copy link
Copy Markdown
Member

@JounQin JounQin commented Mar 12, 2026

Summary by CodeRabbit

  • Chores

    • Downgraded ESLint and related packages from major v10 to v9
    • Removed an unused TypeScript ESLint utility dependency
    • Updated Renovate config to ignore ESLint-related updates
  • Documentation

    • Added a changeset entry documenting a patch release and temporary ESLint downgrade

Copilot AI review requested due to automatic review settings March 12, 2026 09:59
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 12, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 82f2a64c-2bfa-409c-b992-7c7002aeab54

📥 Commits

Reviewing files that changed from the base of the PR and between bd080a5 and 91eca9d.

⛔ Files ignored due to path filters (1)
  • yarn.lock is excluded by !**/yarn.lock, !**/*.lock
📒 Files selected for processing (4)
  • .changeset/eleven-sites-notice.md
  • .renovaterc
  • package.json
  • packages/doom/package.json
🚧 Files skipped from review as they are similar to previous changes (2)
  • .renovaterc
  • package.json

Walkthrough

Downgrades ESLint and @eslint/js from v10 to v9 in root and packages/doom, removes @typescript-eslint/utils from packages/doom, adds a changeset documenting the downgrade, and updates .renovaterc to ignore eslint and @eslint/js.

Changes

Cohort / File(s) Summary
Changeset & Renovate
.changeset/eleven-sites-notice.md, .renovaterc
Adds a changeset noting the temporary ESLint downgrade; adds @eslint/js and eslint to Renovate ignoreDeps.
Root manifest
package.json
Downgrades devDependency eslint from ^10.0.3 to ^9.39.3.
Doom package manifest
packages/doom/package.json
Downgrades @eslint/js ^10.0.1^9.39.3, downgrades eslint ^10.0.3^9.39.3, and removes @typescript-eslint/utils from dependencies.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Suggested labels

dependencies

Poem

🐰
I hopped through package.json with care,
Rolled back versions from ten to nine fair,
A changeset tucked, Renovate told to rest,
Now linting's calmer — for now we jest!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'fix: downgrade to ESLint 9 temporarily' directly reflects the main changes: ESLint is downgraded from version 10 to version 9 across multiple package files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/eslint

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Mar 12, 2026

🦋 Changeset detected

Latest commit: 91eca9d

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@alauda/doom Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Mar 12, 2026

Open in StackBlitz

yarn add https://pkg.pr.new/@alauda/doom@284.tgz
yarn add https://pkg.pr.new/@alauda/doom-export@284.tgz

commit: 91eca9d

Copy link
Copy Markdown
Contributor

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

This PR temporarily downgrades ESLint-related dependencies from v10 to v9 across the repo to restore/maintain compatibility, and prevents Renovate from auto-upgrading them again.

Changes:

  • Downgrade eslint to ^9.39.3 (root) and eslint/@eslint/js to ^9.39.3 in @alauda/doom.
  • Update yarn.lock accordingly for the ESLint v9 dependency graph.
  • Add Renovate ignore rules for eslint and @eslint/js, and add a changeset documenting the temporary downgrade.

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
yarn.lock Lockfile update reflecting the ESLint v9 downgrade and resulting transitive dependency changes.
packages/doom/package.json Downgrades eslint and @eslint/js to v9 ranges (removes the explicit @typescript-eslint/utils dependency).
package.json Downgrades root eslint devDependency to v9 range.
.renovaterc Prevents Renovate from upgrading eslint / @eslint/js (and related ignored deps).
.changeset/eleven-sites-notice.md Records the downgrade as a patch changeset for @alauda/doom.

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

You can also share your feedback on Copilot code review. Take the survey.

@JounQin JounQin merged commit b55d473 into main Mar 12, 2026
14 checks passed
@JounQin JounQin deleted the fix/eslint branch March 12, 2026 10:08
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.

2 participants