Skip to content

CI: optimize build speeds and cache sizes by migrating to Swatinem/rust-cache#5161

Open
RishavTiwari25 wants to merge 2 commits intoboa-dev:mainfrom
RishavTiwari25:ci/migrate-rust-cache
Open

CI: optimize build speeds and cache sizes by migrating to Swatinem/rust-cache#5161
RishavTiwari25 wants to merge 2 commits intoboa-dev:mainfrom
RishavTiwari25:ci/migrate-rust-cache

Conversation

@RishavTiwari25
Copy link
Contributor

This Pull Request fixes/closes #5160

It changes the following:

  • Migrates all 15 manual actions/cache cargo configuration blocks across the repository to the industry standard Swatinem/rust-cache@v2.
  • Cleans up and deletes over 130 lines of redundant caching YAML boilerplate.
  • Automatically prevents 10GB cache storage limit evictions by natively scrubbing and stripping orphaned cargo compiler artifacts using cargo-sweep before zipping them into the cache directory.
  • Dynamically resolves smarter, more granular rust cache keys based on rustc versions, Cargo.lock hashing, and job runners automatically out of the box.

@RishavTiwari25 RishavTiwari25 requested a review from a team as a code owner March 19, 2026 13:13
@github-actions github-actions bot added C-Tests Issues and PRs related to the tests. Waiting On Review Waiting on reviews from the maintainers C-Builtins PRs and Issues related to builtins/intrinsics C-Actions Pull requests that update Github Actions code labels Mar 19, 2026
@github-actions github-actions bot added this to the v1.0.0 milestone Mar 19, 2026
@github-actions
Copy link

github-actions bot commented Mar 19, 2026

Test262 conformance changes

Test result main count PR count difference
Total 52,963 52,963 0
Passed 50,079 50,079 0
Ignored 2,072 2,072 0
Failed 812 812 0
Panics 0 0 0
Conformance 94.55% 94.55% 0.00%

Tested main commit: 26271b7c3f6f357f12fd835a0402d8b643f03db5
Tested PR commit: 334b75a34a398e57b1daf485d3b83c5e3eb62c75
Compare commits: 26271b7...334b75a

@jedel1043
Copy link
Member

Funny LLM with its funny changes

You literally opened a PR to migrate to commit hashes, and now you're introducing rust-cache using tag versioning

@jedel1043
Copy link
Member

Also you left the old cache action there, so now it's not only more expensive, but it's slower

@RishavTiwari25
Copy link
Contributor Author

Hi
You are totally right,
Since i am new to merge conflicts which was shown in my initial commit i pressed the button 'accept both changes' which reverted my changes. Then i accepted merge and commit.
I am sorry about this :)

@jedel1043
Copy link
Member

That doesn't explain why you're still using tag versioning instead of sha versioning, but okay
Also your changes broke the actions, so something's wrong in there

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-Actions Pull requests that update Github Actions code C-Builtins PRs and Issues related to builtins/intrinsics C-Tests Issues and PRs related to the tests. Waiting On Review Waiting on reviews from the maintainers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI: optimize build speeds and cache sizes by migrating to Swatinem/rust-cache

2 participants