Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

ci: Remove actions-rs #4514

Merged
merged 2 commits into from
Apr 25, 2024
Merged

ci: Remove actions-rs #4514

merged 2 commits into from
Apr 25, 2024

Conversation

goatgoose
Copy link
Contributor

@goatgoose goatgoose commented Apr 23, 2024

Resolved issues:

Resolves #4480.

Description of changes:

s2n-tls uses the actions-rs Github action to perform rust-related operations, such as installing the rust toolchain and running clippy. However, actions-rs is no longer being maintained. This PR migrates from actions-rs to just running the rust commands manually.

Call-outs:

None

Testing:

The Rust CI jobs should succeed.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@github-actions github-actions bot added the s2n-core team label Apr 23, 2024
@goatgoose goatgoose changed the title ci: Migrate to rustup from actions-rs ci: Remove actions-rs Apr 23, 2024
@goatgoose goatgoose marked this pull request as ready for review April 23, 2024 23:51
@goatgoose goatgoose requested review from lrstewart and jouho April 23, 2024 23:52
@goatgoose goatgoose enabled auto-merge (squash) April 24, 2024 23:09
@goatgoose goatgoose merged commit f711736 into aws:main Apr 25, 2024
32 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ci: migrate away from actions-rs
3 participants