Skip to content

Conversation

aladdin-add
Copy link
Member

@aladdin-add aladdin-add commented Oct 8, 2025

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
fixes #231

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

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.

Could we also update the Node.js engine versions in plugin/templates/_package.json as well?

"engines": {
"node": "^18.18.0 || ^20.9.0 || >=21.1.0"
},

Or is it intentionally missing?

@lumirlumir lumirlumir moved this from Needs Triage to Implementing in Triage Oct 10, 2025
lumirlumir
lumirlumir previously approved these changes Oct 11, 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, since it's a breaking change.

@lumirlumir lumirlumir moved this from Implementing to Second Review Needed in Triage Oct 11, 2025
@aladdin-add aladdin-add marked this pull request as draft October 14, 2025 07:27
@aladdin-add aladdin-add moved this from Second Review Needed to Blocked in Triage Oct 14, 2025
Co-authored-by: Mike McCready <66998419+MikeMcC399@users.noreply.github.com>
Co-authored-by: 루밀LuMir <rpfos@naver.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Blocked

Development

Successfully merging this pull request may close these issues.

EBADENGINE for yo eslint:plugin under Node.js 18

3 participants