Skip to content

ci: remove most actions-rs actions#191

Closed
robjtede wants to merge 1 commit intoNullus157:prīmumfrom
robjtede:rm-actions-rs
Closed

ci: remove most actions-rs actions#191
robjtede wants to merge 1 commit intoNullus157:prīmumfrom
robjtede:rm-actions-rs

Conversation

@robjtede
Copy link
Copy Markdown
Member

@robjtede robjtede commented May 6, 2023

actions-rs actions are unmaintained.

I'm part of a group maintaining actions-rust-lang actions. The setup-toolchain action has some nice benefits like automatic dependency caching which cuts ~1 minute off the cargo test job. Coverage job is also ~3 mins (2x) faster now.

blocked on #190

@robjtede robjtede marked this pull request as draft May 6, 2023 14:15
@robjtede robjtede force-pushed the rm-actions-rs branch 3 times, most recently from f21e021 to fc0e34f Compare May 6, 2023 15:05
@robjtede robjtede marked this pull request as ready for review May 8, 2023 14:32
@Nemo157
Copy link
Copy Markdown
Member

Nemo157 commented May 8, 2023

bors r+

bors Bot added a commit that referenced this pull request May 8, 2023
191: ci: remove most actions-rs actions r=Nemo157 a=robjtede

`actions-rs` actions are unmaintained.

I'm part of a group maintaining `actions-rust-lang` actions. The setup-toolchain action has some nice benefits like automatic dependency caching which cuts ~1 minute off the `cargo test` job. Coverage job is also ~3 mins (2x) faster now.

blocked on #190 

Co-authored-by: Rob Ede <robjtede@icloud.com>
@bors bors Bot closed this May 8, 2023
@robjtede robjtede deleted the rm-actions-rs branch May 8, 2023 14:55
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.

2 participants