Skip to content

chore: Use swatinem/rust-cache action for caching Rust#102

Merged
kou merged 1 commit into
apache:mainfrom
Benjamin-Philip:bp/ci-rust-cache
Jul 9, 2026
Merged

chore: Use swatinem/rust-cache action for caching Rust#102
kou merged 1 commit into
apache:mainfrom
Benjamin-Philip:bp/ci-rust-cache

Conversation

@Benjamin-Philip

Copy link
Copy Markdown
Collaborator

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.

Copilot AI review requested due to automatic review settings July 8, 2026 16:51
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

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.
See https://github.com/apache/arrow-erlang/blob/main/CONTRIBUTING.md#forks for instructions on how to configure.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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-cache in the Erlang CI workflow.
  • Adds Rust toolchain + swatinem/rust-cache steps to the Dialyzer job (to align Rust caching/installation across jobs that compile).
  • Updates the Docs workflow to use swatinem/rust-cache for 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.

@Benjamin-Philip Benjamin-Philip requested a review from kou July 8, 2026 16:57

@kou kou left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

+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.

@kou kou merged commit 7c26382 into apache:main Jul 9, 2026
12 checks passed
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