-
Notifications
You must be signed in to change notification settings - Fork 898
test: test Node.js 24 in CI #5661
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5661 +/- ##
=======================================
Coverage 95.03% 95.03%
=======================================
Files 310 310
Lines 7992 7992
Branches 1614 1614
=======================================
Hits 7595 7595
Misses 397 397 🚀 New features to boost your workflow:
|
_headers was removed in 57551
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me, but I'll defer to @JamieDanielson on whether it would help to wait on merging this until her larger #5665 (in progress) is finished and merged.
#5665 was merged and there is no conflicts. |
Which problem is this PR solving?
Testing on the latest version of Node.js
Fixes # N/A
Short description of the changes
Node.js v24 was recently released. This change adds it to the CI. (Adapted from 34a2dd5)
Type of change
How Has This Been Tested?
Checklist: