Skip to content

Add eslint-plugin configs#17956

Open
JLHwung wants to merge 2 commits into
babel:mainfrom
JLHwung:add-eslint-plugin-presets
Open

Add eslint-plugin configs#17956
JLHwung wants to merge 2 commits into
babel:mainfrom
JLHwung:add-eslint-plugin-presets

Conversation

@JLHwung
Copy link
Copy Markdown
Contributor

@JLHwung JLHwung commented Apr 28, 2026

Q                       A
Fixed Issues?
Patch: Bug Fix?
Major: Breaking Change?
Minor: New Feature? Yes
Tests Added + Pass? Yes
Documentation PR Link
Any Dependency Changes?
License MIT

This PR includes commits from #17950.

In this PR we add two config presets .recommended and .all for the @babel/eslint-plugin via the .configs property. The naming convention aligns to the official ESLint plugin @eslint/js.

@JLHwung JLHwung added PR: New Feature 🚀 A type of pull request used for our changelog categories PR: Needs Docs area: eslint labels Apr 28, 2026
@babel-bot
Copy link
Copy Markdown
Collaborator

babel-bot commented Apr 28, 2026

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

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Apr 28, 2026

Open in StackBlitz

commit: 801c53d

Comment on lines +4 to +7
export const rules: Record<string, SeverityString> = {
// ESLint core rules
// babel-eslint-plugin rules
};
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is there a reason for this?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good question. The recommended set is empty because none of official ESLint rules that we replaced are in the @eslint/js recommended set. However, if we add new rules in the future, the recommended set may add new rules. The placeholder is here such that we don't have to remove and add back the recommended set.

@JLHwung JLHwung force-pushed the add-eslint-plugin-presets branch 2 times, most recently from 7b5f938 to b18c1d7 Compare May 2, 2026 02:31
@JLHwung JLHwung force-pushed the add-eslint-plugin-presets branch from b18c1d7 to 801c53d Compare May 11, 2026 13:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: eslint PR: Needs Docs PR: New Feature 🚀 A type of pull request used for our changelog categories

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants