Skip to content

Test crates workflow caching#528

Merged
softmarshmallow merged 1 commit intomainfrom
chore/test-crates-workflow-caching-c815
Feb 5, 2026
Merged

Test crates workflow caching#528
softmarshmallow merged 1 commit intomainfrom
chore/test-crates-workflow-caching-c815

Conversation

@softmarshmallow
Copy link
Member

@softmarshmallow softmarshmallow commented Feb 5, 2026

Improve test-crates.yml cache reuse to reduce workflow execution time.

The previous rust-cache configuration frequently missed caches due to hashing Cargo files into the key, leading to long (30+ minute) run times. This change disables that hash, caches all workspace crates, and saves the cache on failure, allowing for better cache hit rates and faster subsequent runs.


Open in Cursor Open in Web

Summary by CodeRabbit

  • Chores
    • Updated CI/CD caching configuration to optimize build performance and reliability.

Co-authored-by: Universe <universe@grida.co>
@cursor
Copy link

cursor bot commented Feb 5, 2026

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@vercel
Copy link

vercel bot commented Feb 5, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
backgrounds Ready Ready Preview, Comment Feb 5, 2026 1:45pm
blog Ready Ready Preview, Comment Feb 5, 2026 1:45pm
docs Ready Ready Preview, Comment Feb 5, 2026 1:45pm
grida Ready Ready Preview, Comment Feb 5, 2026 1:45pm
viewer Ready Ready Preview, Comment Feb 5, 2026 1:45pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
code Ignored Ignored Feb 5, 2026 1:45pm
legacy Ignored Ignored Feb 5, 2026 1:45pm

Request Review

@coderabbitai
Copy link

coderabbitai bot commented Feb 5, 2026

Walkthrough

The test-crates GitHub Actions workflow was updated with three new cache configuration parameters for the Swatinem/rust-cache action: disabling the Rust environment hash key, enabling workspace crates caching, and enabling cache persistence on build failure.

Changes

Cohort / File(s) Summary
GitHub Actions Cache Configuration
.github/workflows/test-crates.yml
Added three new inputs to the rust-cache action: add-rust-environment-hash-key: false, cache-workspace-crates: true, and cache-on-failure: true, modifying cache computation and retention behavior.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A cache so swift, with settings refined,
No hash keys weighing down the bind,
Workspace crates all bundled tight,
Failure brings persistence's might!
Our builds now bounce with hoppy glee,

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main change: configuring caching in the test-crates workflow to improve cache reuse and reduce execution time.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/test-crates-workflow-caching-c815

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@softmarshmallow softmarshmallow merged commit feb62e5 into main Feb 5, 2026
13 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.

2 participants