-
Notifications
You must be signed in to change notification settings - Fork 42
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 Node 20 support #170
Conversation
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.
looks like you may have forgot to rebase before adding this. there's a conflict with publish-layers.sh
@bizob2828 I do have to rebase, though I didn't forget: this branch was opened before I removed Node 14 support today. |
fc02962
to
432da66
Compare
Noting that I need to add check runs for Node 20. EDIT: added, but currently failing integration for Node 20 in CI (not locally). |
CI for Node 20 is currently blocked until serverless-offline releases its just-merged support for Node 20: https://github.com/dherault/serverless-offline |
c508ef4
to
13387bc
Compare
@bizob2828 I've rebased this against the wrapper-update PR, and added the new serverless-offline with Node 20 support, so now we're passing. |
Signed-off-by: mrickard <[email protected]>
c820a87
to
5592771
Compare
PR to anticipate Node 20 being introduced as a supported Lambda runtime.
Closes #169
Closes NR-141663