Skip to content

ci: bump actions/checkout to v4 - #685

Merged
CarsonRoscoe merged 1 commit into
coinbase:mainfrom
dizer-ti:main
Apr 28, 2025
Merged

CarsonRoscoe merged 1 commit into
coinbase:mainfrom
dizer-ti:main

Conversation

@dizer-ti

Copy link
Copy Markdown
Contributor

Hey ! GitHub‑hosted runners now use Node 20, so checkout v4 is required
This PR updates all workflows to actions/checkout@v4 , no functional changes expected

@dizer-ti
dizer-ti requested a review from murrlincoln as a code owner April 22, 2025 18:45
@cb-heimdall

cb-heimdall commented Apr 22, 2025 •

Copy link
Copy Markdown

✅ Heimdall Review Status

Requirement Status More Info
Reviews ✅ 1/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@CarsonRoscoe CarsonRoscoe left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you, appreciate the catch :)

@CarsonRoscoe CarsonRoscoe left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ah apologies @dizer-ti, our Repo only accepts signed commits.
See here for instructions: https://docs.github.com/en/authentication/managing-commit-signature-verification/about-commit-signature-verification

Once setup, re-apply the commit via rebasing against main, or running git reset --soft HEAD~1 and re-committing the code.

@dizer-ti

Copy link
Copy Markdown
Contributor Author

@CarsonRoscoe ready

@CarsonRoscoe
CarsonRoscoe merged commit 22f5fdc into coinbase:main Apr 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants