Skip to content

Comments

fix: use types from @eslint/plugin-kit and update @eslint/core#607

Merged
lumirlumir merged 3 commits intomainfrom
refactor/plugin-kit-types
Feb 15, 2026
Merged

fix: use types from @eslint/plugin-kit and update @eslint/core#607
lumirlumir merged 3 commits intomainfrom
refactor/plugin-kit-types

Conversation

@Pixel998
Copy link
Contributor

@Pixel998 Pixel998 commented Jan 13, 2026

Prerequisites checklist

What is the purpose of this pull request?

To align the codebase with the type changes introduced in @eslint/plugin-kit v0.6.0 by using the new canonical type exports and removing reliance on deprecated types from @eslint/core.

What changes did you make? (Give an overview)

  • Updated type imports so CustomRuleTypeDefinitions and CustomRuleDefinitionType come from @eslint/plugin-kit instead of @eslint/core.
  • Replaced the locally defined WithExit helper type with the new CustomRuleVisitorWithExit type provided by @eslint/plugin-kit.

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

@eslintbot eslintbot added this to Triage Jan 13, 2026
@github-project-automation github-project-automation bot moved this to Needs Triage in Triage Jan 13, 2026
@Pixel998 Pixel998 moved this from Needs Triage to Implementing in Triage Jan 13, 2026
@Pixel998 Pixel998 self-assigned this Jan 13, 2026
@Pixel998 Pixel998 marked this pull request as ready for review February 15, 2026 09:57
Copy link
Contributor

@snitin315 snitin315 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 it open for others to review.

@snitin315 snitin315 moved this from Implementing to Second Review Needed in Triage Feb 15, 2026
@lumirlumir lumirlumir changed the title refactor: use types from @eslint/plugin-kit fix: use types from @eslint/plugin-kit and update @eslint/core Feb 15, 2026
Copy link
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. Some changes overlap with #593, but there are additional considerations there. I’ll merge this first and narrow the scope of #593.

Also, I’m marking this PR as a fix since it updates dependencies.

@lumirlumir lumirlumir merged commit f5d4ce0 into main Feb 15, 2026
24 checks passed
@lumirlumir lumirlumir deleted the refactor/plugin-kit-types branch February 15, 2026 13:43
@github-project-automation github-project-automation bot moved this from Second Review Needed to Complete in Triage Feb 15, 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.

3 participants