Skip to content

Update actions/checkout in GitHub Actions workflows to v3 #294

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

Merged
merged 1 commit into from
Dec 29, 2022
Merged

Update actions/checkout in GitHub Actions workflows to v3 #294

merged 1 commit into from
Dec 29, 2022

Conversation

striezel
Copy link

Updates the actions/checkout action used in the GitHub Actions workflow to its newest major version.

Changes in actions/checkout:

v3.1.0

  • Use @actions/core saveState and getState
  • Add github-server-url input

v3.0.2

  • Add input set-safe-directory

v3.0.1

  • Fixed an issue where checkout failed to run in container jobs due to the new git setting safe.directory
  • Bumped various npm package versions

v3.0.0

  • Update to node 16

Still using v2 of actions/checkout will generate some warning like in this run: https://github.com/servo/rust-smallvec/actions/runs/3783768281

Node.js 12 actions are deprecated. For more information see: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/. Please update the following actions to use Node.js 16: actions/checkout@v2, actions-rs/toolchain@v1

The PR will get rid of those warnings for actions/checkout, because v3 uses Node.js 16.

@jdm
Copy link
Member

jdm commented Dec 29, 2022

@bors-servo r+

@bors-servo
Copy link
Contributor

📌 Commit 49b1e64 has been approved by jdm

@bors-servo
Copy link
Contributor

⌛ Testing commit 49b1e64 with merge b6583d6...

@bors-servo
Copy link
Contributor

☀️ Test successful - checks-github
Approved by: jdm
Pushing b6583d6 to master...

@bors-servo bors-servo merged commit b6583d6 into servo:master Dec 29, 2022
@striezel striezel deleted the actions-checkout-update branch December 29, 2022 20:09
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.

3 participants