Skip to content
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

Replace unmaintained actions-rs/toolchain action in CI workflows #454

Merged

Conversation

striezel
Copy link
Contributor

@striezel striezel commented Mar 2, 2023

Basically all of the actions-rs/* actions are unmaintained. See actions-rs/toolchain#216 for more information. Due to their age they generate several warnings in CI runs, for example in https://github.com/RustCrypto/hashes/actions/runs/3716377913:

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-rs/toolchain@v1

To get rid of some of those warnings the occurrences of actions-rs/toolchain are replaced by dtolnay/rust-toolchain.

@tarcieri
Copy link
Member

tarcieri commented Mar 2, 2023

Thanks. We've been making this change elsewhere (e.g. RustCrypto/formats#887) but there are quite a few repos to update

@tarcieri tarcieri requested a review from newpavlov March 2, 2023 23:18
@tarcieri
Copy link
Member

tarcieri commented Mar 2, 2023

Hmm, I think GitHub Actions is having some issues... jobs are showing up as cancelled but without failures

@tarcieri
Copy link
Member

tarcieri commented Mar 3, 2023

@striezel can you commit --amend and try re-pushing? CI seems wedged

Basically all of the `actions-rs/*` actions are unmaintained. See
<actions-rs/toolchain#216> for more
information. Due to their age they generate several warnings in
CI runs.

To get rid of some of those warnings the occurrences of
`actions-rs/toolchain` are replaced by `dtolnay/rust-toolchain`.
@striezel striezel force-pushed the actions-rs-toolchain-replacement branch from d32c98e to daa1894 Compare March 3, 2023 23:05
@striezel
Copy link
Contributor Author

striezel commented Mar 3, 2023

@striezel can you commit --amend and try re-pushing? CI seems wedged

Done and force pushed. And while I was at it, I also replaced the remaining target: ... with targets: ... for consistency.

@newpavlov newpavlov merged commit b059d4b into RustCrypto:master Mar 3, 2023
@newpavlov
Copy link
Member

Thank you!

@striezel striezel deleted the actions-rs-toolchain-replacement branch March 3, 2023 23:34
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