Skip to content

Conversation

@aladdin-add
Copy link
Member

Prerequisites checklist

What is the purpose of this pull request?

Updated package.json of the package to require:

"node": "^20.19.0 || ^22.13.0 || >=24"

This drops support for Node.js 18.

What changes did you make? (Give an overview)

Related Issues

refs eslint/eslint#19969

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

lumirlumir
lumirlumir previously approved these changes Oct 14, 2025
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, thanks!

Would like @eslint/eslint-tsc to verify and merge, since it's a breaking change.

@lumirlumir lumirlumir added the accepted There is consensus among the team that this change meets the criteria for inclusion label Oct 14, 2025
Copy link
Member

@fasttime fasttime 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!

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 breaking feature

Projects

Status: Blocked

Development

Successfully merging this pull request may close these issues.

4 participants