Skip to content

Conversation

@MarijnS95
Copy link
Member

GitHub Actions' runner-images already come fully loaded with all the Rust tools that we need 1: remove the bloated and outdated actions-rs/toolchain setup step for that, and also remove the actions-rs/cargo action that only serves as a more convoluted way to run simple cargo shell commands.

Besides, actions-rs are unmaintained and outdated, and throw an alarming number of spammy warnings in CI build output.

GitHub Actions' `runner-images` already come fully loaded with all the
Rust tools that we need [1]: remove the bloated and outdated
`actions-rs/toolchain` setup step for that, and also remove the
`actions-rs/cargo` action that only serves as a more convoluted way to
run simple `cargo` shell commands.

Besides, `actions-rs` are unmaintained and outdated, and throw an
alarming number of spammy warnings in CI build output.

[1]: https://github.com/actions/runner-images/blob/main/images/linux/Ubuntu2204-Readme.md#rust-tools
Copy link
Member

@Jasper-Bekkers Jasper-Bekkers left a comment

Choose a reason for hiding this comment

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

Much better.

@MarijnS95 MarijnS95 merged commit 890786b into main Nov 3, 2023
@MarijnS95 MarijnS95 deleted the yeet-actions-rs branch November 3, 2023 18:37
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