Skip to content

Conversation

Pixel998
Copy link
Contributor

@Pixel998 Pixel998 commented Sep 22, 2025

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:

What changes did you make? (Give an overview)

This PR temporarily pins the jiti dependency to version 2.5.1 to fix unit tests which are currently failing with jiti v2.6.0. E.g.:

https://github.com/eslint/eslint/actions/runs/17924949525/job/50968611432?pr=20149

Refs unjs/jiti#406

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

@Pixel998 Pixel998 requested a review from a team as a code owner September 22, 2025 20:36
@github-project-automation github-project-automation bot moved this to Needs Triage in Triage Sep 22, 2025
@eslint-github-bot eslint-github-bot bot added the build This change relates to ESLint's build process label Sep 22, 2025
Copy link

netlify bot commented Sep 22, 2025

Deploy Preview for docs-eslint canceled.

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

Copy link
Member

@aladdin-add aladdin-add 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!

@aladdin-add aladdin-add merged commit bb99566 into eslint:main Sep 23, 2025
30 checks passed
@github-project-automation github-project-automation bot moved this from Needs Triage to Complete in Triage Sep 23, 2025
@Pixel998 Pixel998 mentioned this pull request Sep 23, 2025
1 task
@mdjermanovic mdjermanovic mentioned this pull request Sep 24, 2025
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build This change relates to ESLint's build process

Projects

Status: Complete

Development

Successfully merging this pull request may close these issues.

2 participants