Skip to content

chore: Update runtime version to 20 for appengine sample#3643

Closed
heypisha wants to merge 4 commits intoGoogleCloudPlatform:mainfrom
heypisha:patch-1
Closed

chore: Update runtime version to 20 for appengine sample#3643
heypisha wants to merge 4 commits intoGoogleCloudPlatform:mainfrom
heypisha:patch-1

Conversation

@heypisha
Copy link
Copy Markdown

nodejs16 runtime is outdated and throws an error that its support has expired. Updating to the latest, nodejs20, that has support through 2026-04-30.

Description

Fixes #3642

Note: Before submitting a pull request, please open an issue for discussion if you are not associated with Google.

Checklist

  • I have followed guidelines from CONTRIBUTING.MD and Samples Style Guide
  • Tests pass: npm test (see Testing)
  • Lint pass: npm run lint (see Style)
  • These samples need a new API enabled in testing projects to pass (let us know which ones)
  • These samples need a new/updated env vars in testing projects set to pass (let us know which ones)
  • This pull request is from a branch created directly off of GoogleCloudPlatform/nodejs-docs-samples. Not a fork.
  • This sample adds a new sample directory, and I updated the CODEOWNERS file with the codeowners for this sample
  • This sample adds a new sample directory, and I created GitHub Actions workflow for this sample
  • This sample adds a new Product API, and I updated the Blunderbuss issue/PR auto-assigner with the codeowners for this sample
  • Please merge this PR for me once it is approved

nodejs16 runtime is outdated and throws an error that its support has expired. Updating to the latest, nodejs20, that has support through 2026-04-30.
@heypisha heypisha requested review from a team as code owners March 13, 2024 20:12
@google-cla
Copy link
Copy Markdown

google-cla Bot commented Mar 13, 2024

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@product-auto-label product-auto-label Bot added samples Issues that are directly related to samples. api: appengine Issues related to the App Engine Admin API API. asset: pattern DEE Asset tagging - Pattern. labels Mar 13, 2024
@iennae iennae changed the title Update runtime version to 20 chore: Update runtime version to 20 for appengine sample Apr 17, 2024
@iennae
Copy link
Copy Markdown
Contributor

iennae commented Apr 17, 2024

@pierrettemugisha appreciate you creating this quick fix. I've gone ahead and replaced with a larger update to app engine samples for standard environment with #3657 . As I can branch the repository, tests can run within the PR. I'm going to go ahead and close this PR.

@iennae iennae closed this Apr 17, 2024
gcf-merge-on-green Bot pushed a commit that referenced this pull request Apr 17, 2024
Node.js 10, 12, 14, and 16 have reached end of
support on January 30, 2024. Existing
applications using these versions will continue
to run and receive traffic. This update to
samples ensures new deploys will work as
expected.

## Description

Fixes #3645 
Fixes #3642 
Replaces #3643 

## Checklist

- [X] Please **merge** this PR for me once it is approved
@heypisha heypisha deleted the patch-1 branch April 17, 2024 15:30
@heypisha heypisha restored the patch-1 branch April 17, 2024 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: appengine Issues related to the App Engine Admin API API. asset: pattern DEE Asset tagging - Pattern. samples Issues that are directly related to samples.

Projects

None yet

4 participants