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?

@coveralls
Copy link

coveralls commented Oct 14, 2025

Pull Request Test Coverage Report for Build 18488516317

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 98.618%

Totals Coverage Status
Change from base Build 18427158029: 0.0%
Covered Lines: 11829
Relevant Lines: 11961

💛 - Coveralls

@aladdin-add aladdin-add moved this from Needs Triage to Blocked in Triage Oct 15, 2025
- 22
- 24
- 25
node: [25.x, 24.x, 22.x, 20.x, "20.19.0"]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think PRIMARY_NODEJS_VERSION should be also changed to match one of the values in this list.

PRIMARY_NODEJS_VERSION: 20

That could fix the CI failure, or result in a different error.

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.

4 participants