Skip to content

Conversation

fasttime
Copy link
Member

Prerequisites checklist

What is the purpose of this pull request? (put an "X" next to an item)

[ ] Documentation update
[ ] Bug fix (template)
[ ] New rule (template)
[ ] Changes an existing rule (template)
[ ] Add autofix to a rule
[ ] Add a CLI option
[ ] Add something to the core
[x] Other, please explain:

Fix types

refs #18771

What changes did you make? (Give an overview)

Added before and after properties to test case base type (RuleTester.ValidTestCase). Support for before and after in test cases was added in ESLint v9.12.0 following the proposal in this RFC document, but the types were not updated.

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

@github-project-automation github-project-automation bot moved this to Needs Triage in Triage Aug 28, 2025
@eslint-github-bot eslint-github-bot bot added the bug ESLint is working incorrectly label Aug 28, 2025
Copy link

netlify bot commented Aug 28, 2025

Deploy Preview for docs-eslint canceled.

Name Link
🔨 Latest commit 0c23555
🔍 Latest deploy log https://app.netlify.com/projects/docs-eslint/deploys/68afdeac9fe082000883d58e

@fasttime fasttime added the types Related to TypeScript types label Aug 28, 2025
@fasttime fasttime marked this pull request as ready for review August 28, 2025 05:19
@fasttime fasttime requested a review from a team as a code owner August 28, 2025 05:19
@mdjermanovic mdjermanovic added the accepted There is consensus among the team that this change meets the criteria for inclusion label Aug 28, 2025
@mdjermanovic mdjermanovic moved this from Needs Triage to Implementing in Triage Aug 28, 2025
Copy link
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!

@mdjermanovic mdjermanovic merged commit dfd11de into main Aug 28, 2025
30 checks passed
@mdjermanovic mdjermanovic deleted the update-test-case-types branch August 28, 2025 07:43
@github-project-automation github-project-automation bot moved this from Implementing to Complete in Triage Aug 28, 2025
nzakas pushed a commit that referenced this pull request Sep 23, 2025
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 bug ESLint is working incorrectly types Related to TypeScript types

Projects

Status: Complete

Development

Successfully merging this pull request may close these issues.

2 participants