Skip to content

refactor: use types from @eslint/plugin-kit - #360

Merged
fasttime merged 2 commits into
mainfrom
refactor/plugin-kit-types
Jan 31, 2026
Merged

fasttime merged 2 commits into
mainfrom
refactor/plugin-kit-types

Conversation

@Pixel998

@Pixel998 Pixel998 commented Jan 13, 2026

Copy link
Copy Markdown
Contributor

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?

@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 January 31, 2026 02:50
@fasttime fasttime added the accepted There is consensus among the team that this change meets the criteria for inclusion label Jan 31, 2026

@fasttime fasttime left a comment

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.

LGTM, thanks!

@fasttime
fasttime merged commit be5454c into main Jan 31, 2026
33 checks passed
@github-project-automation github-project-automation Bot moved this from Implementing to Complete in Triage Jan 31, 2026
@fasttime
fasttime deleted the refactor/plugin-kit-types branch January 31, 2026 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

accepted There is consensus among the team that this change meets the criteria for inclusion chore

Projects

Status: Complete

Development

Successfully merging this pull request may close these issues.

2 participants