Skip to content
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

Bump to Node20 #72

Merged
merged 2 commits into from
Apr 10, 2024
Merged

Conversation

nicholasphair
Copy link
Contributor

Issue #71

Description of changes:
Bumps Node runtime to version 20.

I deployed to my account with no problem. Sorry I can't provide more convincing tests for this pr. Still, hope it saves you some time.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@nicholasphair nicholasphair requested a review from a team as a code owner April 5, 2024 03:36
@nicholasphair nicholasphair changed the title Bump to Node20 (resolves #71) Bump to Node20 Apr 5, 2024
@ConnorKirk
Copy link
Contributor

Thanks @nicholasphair for your contribution.
I needed to upgrade one function to use the aws-js-sdk-v3 as well, as the v2 sdk is no longer present in the node 20 runtime.

@ConnorKirk ConnorKirk merged commit bd2d4b3 into aws-samples:master Apr 10, 2024
1 check passed
@nicholasphair
Copy link
Contributor Author

Oops, sorry I missed that. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants