chore: Use swatinem/rust-cache action for caching Rust#102
Conversation
|
Preview URL: https://Benjamin-Philip.github.io/arrow-erlang-1 If the preview URL doesn't work, you may have forgotten to configure your fork repository for preview. |
There was a problem hiding this comment.
Pull request overview
This PR updates the GitHub Actions workflows to use swatinem/rust-cache for Rust dependency/build caching, replacing the previous manual actions/cache configuration, to reduce CI time spent recompiling Rust NIF code.
Changes:
- Replaces manual Cargo directory caching with
swatinem/rust-cachein the Erlang CI workflow. - Adds Rust toolchain +
swatinem/rust-cachesteps to the Dialyzer job (to align Rust caching/installation across jobs that compile). - Updates the Docs workflow to use
swatinem/rust-cachefor the Rust workspace used during docs generation.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| .github/workflows/erlang.yml | Switches Rust caching to swatinem/rust-cache and ensures Rust is installed/cached for jobs that compile/dialyze. |
| .github/workflows/docs.yml | Switches Rust caching to swatinem/rust-cache for the docs build job. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
kou
left a comment
There was a problem hiding this comment.
+1
BTW, could you open an issue before you work on something? It's for openness. If we share what we want to do publicly, others may be able to join this project.
What's Changed
Uses swatinem/rust-cache for effectively caching Rust. Removes any manual
caching. This will potentially lead to faster CIs since compiling Rust is the
most time intensive common step.