Skip to content

Conversation

@kbdharun
Copy link

@kbdharun kbdharun commented Sep 5, 2023

Description and Context:

Node 16 reaches the end of life soon on 11 Sep 2023. This PR updates the default runtime to node20 (Node 20). I have also bumped the actions/checkout version to v4 for the same.

Related issue:

actions/runner#2732

Closes #1260


A major version bump might be needed after the PRs merge.

@kbdharun kbdharun requested a review from a team as a code owner September 5, 2023 04:27
@chenrui333
Copy link

make sense!

@erikburt
Copy link

Closes #1260.

@todgru todgru mentioned this pull request Jan 26, 2024
10 tasks
@muzimuzhi
Copy link

Bumping to node20 has been done in #1284 and shipped with v4.0.0. This PR needs an update to only bump actions/checkout to v4.

@kbdharun kbdharun changed the title feat: bump to use node20 runtime, actions/checkout to v4 Feb 2, 2024
uses: actions/checkout@v4
- name: Setup Node.js 20.x
uses: actions/setup-node@v3
uses: actions/setup-node@v4

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
uses: actions/setup-node@v4
uses: actions/setup-node@v3

Somehow actions/setup-node is also bumped in the last commit dbe5320. But this change didn't belong to the current default branch (in upstream repo), thus I propose to revert it.

- name: Setup Node.js 20.x
uses: actions/setup-node@v3

Copy link
Author

@kbdharun kbdharun Feb 2, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, which resolving the conflict I seem to have accidentally bumped it as the latest Node 20-based version is v4 series (reverted it, but I don't currently see other PRs bumping it). I personally think setting dependabot for updating GitHub actions would work the best for the current usage.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder why this repo didn't use a dependabot.yml to keep GitHub Actions up-to-date.

Co-authored-by: Yukai Chou <muzimuzhi@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

6 participants