close
Skip to content

feat: bump actions/checkout to v4#1231

Open
kbdharun wants to merge 4 commits into
actions:mainfrom
kbdharun:main
Open

feat: bump actions/checkout to v4#1231
kbdharun wants to merge 4 commits into
actions:mainfrom
kbdharun:main

Conversation

@kbdharun
Copy link
Copy Markdown

@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
Copy Markdown

make sense!

@erikburt
Copy link
Copy Markdown

Closes #1260.

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

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 feat: bump actions/checkout to v4 Feb 2, 2024
Comment thread .github/workflows/workflow.yml Outdated
uses: actions/checkout@v4
- name: Setup Node.js 20.x
uses: actions/setup-node@v3
uses: actions/setup-node@v4
Copy link
Copy Markdown

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
Copy Markdown
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.

Copy link
Copy Markdown

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Upgrade action to use node20

6 participants