Skip to content

Support Node 24#2587

Merged
RobinTail merged 13 commits intomasterfrom
support-node-24
May 6, 2025
Merged

Support Node 24#2587
RobinTail merged 13 commits intomasterfrom
support-node-24

Conversation

@RobinTail
Copy link
Copy Markdown
Owner

@RobinTail RobinTail commented Apr 27, 2025

This will replace #2106
It's still not yet released due to external issue with upgrading v8

Release date shifted to 06.05.2025
nodejs/Release#1081 (comment)

@RobinTail RobinTail added enhancement New feature or request CI/CD labels Apr 27, 2025
@coveralls-official
Copy link
Copy Markdown

coveralls-official Bot commented Apr 27, 2025

Coverage Status

coverage: 100.0%. remained the same
when pulling b4f4e32 on support-node-24
into 872c33f on master.

Comment thread .github/workflows/node.js.yml Outdated
Comment thread .github/workflows/node.js.yml Outdated
Comment thread express-zod-api/package.json Outdated
@RobinTail RobinTail mentioned this pull request May 3, 2025
@RobinTail
Copy link
Copy Markdown
Owner Author

@RobinTail RobinTail marked this pull request as ready for review May 6, 2025 16:11
@socket-security
Copy link
Copy Markdown

socket-security Bot commented May 6, 2025

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatedeslint-plugin-allowed-dependencies@​1.2.0 ⏵ 1.3.0100 +34100100 +3100 +16100

View full report

@RobinTail RobinTail merged commit 7939024 into master May 6, 2025
14 checks passed
@RobinTail RobinTail deleted the support-node-24 branch May 6, 2025 16:14
RobinTail added a commit that referenced this pull request May 6, 2025
@RobinTail RobinTail mentioned this pull request May 12, 2025
RobinTail added a commit that referenced this pull request Jun 13, 2025
This should be default when `@types/node` get updated to v24.
Related to #2587 
Related to #2619 

The breaking change is that it requires Node 20.18.1, while it works
fine with 20.9.0.
The dependency, however, is only used for testing, so it's still a
questionable move.

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

- **Chores**
- Updated Node.js engine requirements to a newer minimum version in the
20.x range.
	- Upgraded the "undici" development dependency to the latest version.
	- Adjusted CI workflow to test against the updated Node.js version.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/CD enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant