-
Notifications
You must be signed in to change notification settings - Fork 51
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
feat: Added support for Node 20 #364
Conversation
ea8f2ab
to
b43a5a8
Compare
Signed-off-by: mrickard <[email protected]>
b43a5a8
to
661306e
Compare
Signed-off-by: mrickard <[email protected]>
661306e
to
3a3cd8e
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #364 +/- ##
=======================================
Coverage 33.08% 33.08%
=======================================
Files 4 4
Lines 535 535
Branches 145 145
=======================================
Hits 177 177
Misses 330 330
Partials 28 28
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Signed-off-by: mrickard <[email protected]>
Signed-off-by: mrickard <[email protected]>
Signed-off-by: mrickard <[email protected]>
Signed-off-by: mrickard <[email protected]>
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.
Can we also add this to our list to fix? This feels very rigid to update post release
Yes--I'll make a ticket for that. |
Hi guys 👋🏼 This is great! Will there be a tag and release for this merge? |
@bakeruk We've just released v4.3.0, which includes support for Node 20. Thank you! |
Added support for the newly-released Node 20 layers, including adding a specific test fixture showing layers for each Node version.
Updated semver to 7.5.4 (pulling from a stale update PR)
Closes #363
Closes NR-141651