Skip to content

Remove unused code for old ESLint versions#17766

Merged
nicolo-ribaudo merged 2 commits intobabel:mainfrom
liuxingbaoyu:remove-old-eslint
Feb 13, 2026
Merged

Remove unused code for old ESLint versions#17766
nicolo-ribaudo merged 2 commits intobabel:mainfrom
liuxingbaoyu:remove-old-eslint

Conversation

@liuxingbaoyu
Copy link
Copy Markdown
Member

@liuxingbaoyu liuxingbaoyu commented Jan 31, 2026

Q                       A
Fixed Issues? Fixes #1, Fixes #2
Patch: Bug Fix?
Major: Breaking Change?
Minor: New Feature?
Tests Added + Pass? Yes
Documentation PR Link babel/website#3177
Any Dependency Changes?
License MIT

@liuxingbaoyu
Copy link
Copy Markdown
Member Author

Unfortunately, we forgot to change the eslint-plugin.
Besides the peer dependency version, three other rules are no longer needed.

@babel-bot
Copy link
Copy Markdown
Collaborator

babel-bot commented Jan 31, 2026

Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/60840

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Jan 31, 2026

Open in StackBlitz

commit: b1a376f

@liuxingbaoyu liuxingbaoyu added PR: Breaking Change 💥 A type of pull request used for our changelog categories for next major release area: eslint PR: Needs Docs labels Feb 4, 2026
@nicolo-ribaudo nicolo-ribaudo changed the title Remove unused ESLint code Remove unused code for old ESLint versions Feb 13, 2026
@nicolo-ribaudo
Copy link
Copy Markdown
Member

Given that we already mentioned in the migration docs that people need to use ESLint 9, so we need more docs?

@nicolo-ribaudo nicolo-ribaudo merged commit b6c765e into babel:main Feb 13, 2026
54 checks passed
@JLHwung JLHwung mentioned this pull request Feb 13, 2026
NadirAli674

This comment was marked as spam.

NadirAli674

This comment was marked as spam.

@JLHwung
Copy link
Copy Markdown
Contributor

JLHwung commented Feb 23, 2026

Given that we already mentioned in the migration docs that people need to use ESLint 9, so we need more docs?

@liuxingbaoyu Can you update the migration docs? Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: eslint PR: Breaking Change 💥 A type of pull request used for our changelog categories for next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants