Skip to content

feat: add languages and docs.dialects to rule meta types#421

Merged
lumirlumir merged 3 commits intomainfrom
add-languages
Apr 3, 2026
Merged

feat: add languages and docs.dialects to rule meta types#421
lumirlumir merged 3 commits intomainfrom
add-languages

Conversation

@fasttime
Copy link
Copy Markdown
Member

@fasttime fasttime commented Apr 1, 2026

Prerequisites checklist

AI acknowledgment

  • I did not use AI to generate this PR.
  • (If the above is not checked) I have reviewed the AI-generated content before submitting.

What is the purpose of this pull request?

Update the core types.

What changes did you make? (Give an overview)

This PR adds the languages property to the RulesMeta interface (the type used for rule meta objects), and dialects to RulesMetaDocs (the meta.docs type), following the design described in the rule languages RFC. The related change in ESLint has already been merged in eslint/eslint#20571. It also deprecates language and dialects in RulesMeta.

Related Issues

refs eslint/eslint#19462

Is there anything you'd like reviewers to focus on?

@github-project-automation github-project-automation Bot moved this to Needs Triage in Triage Apr 1, 2026
@fasttime fasttime marked this pull request as ready for review April 1, 2026 18:11
Comment thread packages/core/src/types.ts
@nzakas nzakas moved this from Needs Triage to Implementing in Triage Apr 1, 2026
Comment thread packages/core/src/types.ts
fasttime and others added 2 commits April 2, 2026 22:44
Co-authored-by: 루밀LuMir <rpfos@naver.com>
@fasttime fasttime changed the title feat: add languages to RulesMeta interface feat: add languages and docs.dialects to rule meta types Apr 2, 2026
Copy link
Copy Markdown
Member

@mdjermanovic mdjermanovic left a comment

Choose a reason for hiding this comment

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

LGTM, thanks! Leaving open until tomorrow's release if anyone else would like to review it.

Copy link
Copy Markdown
Contributor

@Pixel998 Pixel998 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@lumirlumir lumirlumir moved this from Implementing to Second Review Needed in Triage Apr 3, 2026
Copy link
Copy Markdown
Member

@lumirlumir lumirlumir left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

Since there are already two approvals, I'll go ahead and merge this PR.

@lumirlumir lumirlumir merged commit 7680f69 into main Apr 3, 2026
32 checks passed
@lumirlumir lumirlumir deleted the add-languages branch April 3, 2026 10:20
@github-project-automation github-project-automation Bot moved this from Second Review Needed to Complete in Triage Apr 3, 2026
@eslintbot eslintbot mentioned this pull request Apr 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Complete

Development

Successfully merging this pull request may close these issues.

5 participants