bevy-website
bevy-website copied to clipboard
Always install Rust and use `Leafwing-Studios/cargo-cache`
This PR has two parts:
- Some jobs don't actually install Rust before using it. This doesn't error because Github's hosted runners come pre-installed with Rust, though it is frequently outdated.
- Use
Leafwing-Studios/cargo-cachefor all jobs that use Rust. This action is wonderful: I've been using it successfully for a month or so now. (Also it's by our friends at Leafwing, so it can be trusted!) This should speed up many jobs without the usual tradeoffs of caching.
I'm marking this as contentious because it is pulling in an outside dependency, but I feel like it's from a trusted source. If this gets major pushback, we can fork Leafwing-Studios/cargo-cache under the Bevy organization and frequently update it.
I'm rebasing this on top of #1403 so CI passes. Consider this blocked until that PR is merged.
Just linking in discussion from the twin PR for the bevy repo. https://github.com/bevyengine/bevy/pull/13040
Yeah no objection to handing off ownership to the Bevy org if this appeals to y'all :)
@mockersf what are your thoughts on merging this? It would provide a good testing ground for introducing this action to the main Bevy repository, and Alice even offered transferring the repository.
Note that cargo-cache is about to release v2: https://github.com/Leafwing-Studios/cargo-cache/pull/17
Let me enable the sweep-cache option before this gets merged :)
Hah! Turns out this found a bug with cargo-sweep: https://github.com/BD103/cargo-sweep/issues/7. Let me fix that real quick...
I'm going to close this. I no longer have the time / energy to work on it, though anyone else is free to pick it up!
I'll create an issue for this, though, so it's not lost.