Repository navigation
Trim redundant CI work - #498
Conversation
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: 67492266-1cdd-48ad-a896-dba42f920bff) |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
📒 Files selected for processing (3)
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughDependabot groups Cargo and GitHub Actions updates by ecosystem. CI Rust cache saves are limited to runs on the main branch. The unsigned application build skips execution when the release-cache restore has an exact hit. ChangesDependabot update grouping
Workflow cache behavior
Priority: ➖ Normal Estimated code review effort: 2 (Simple) | ~10 minutes Change: Other Merge Risk: ⚪ Minimal · up to These changes group dependency updates and reduce redundant cache work without removing PR cache restores or the release build. No actionable merge-blocking risk is evident. Architecture SummaryArchitecture risk: 🔵 Low · up to The changed surface does not map to a changed system, dependency edge, entrypoint, or external dependency. Changed systems: None identified. Architecture concerns Review detailsBefore / after behavior
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
ceiling | e9c66cf | Commit Preview URL Branch Preview URL |
Oct 06 2026, 04:03 AM |
What changes
ci.yml: bothSwatinem/rust-cachesteps getsave-ifon main only. PRs still restore main's cache but no longer write new entries.dependabot.yml: cargo and github-actions updates are grouped, so each run opens one PR per ecosystem instead of one per dependency.warm-release-cache.yml: the Windows release build is skipped when the restore step reports an exact cache hit. The restore still runs, so a hit still refreshes the entry and keeps it from idle eviction. A lock or toolchain change changes the key, misses, and builds and saves as before.Why
The warm-release-cache job ran a 12-19 min Windows build on exact hits, where the save is skipped, so it proved and stored nothing. PR-ref cache saves also crowd the 10 GB cache quota. See the CI audit findings F1 and F2.
Still runs
FrontendandRustchecks are unchanged (same job names, no path filters).release.ymlis untouched. The warm job still checks out, sets up toolchains and computes the key on every trigger, and builds and saves on a miss.Note
Low Risk
CI and Dependabot configuration only; test and release workflows still run on cache miss or non-main branches as before.
Overview
Reduces redundant CI and Dependabot churn without changing what actually gates merges.
Rust CI cache writes are limited to
main: bothSwatinem/rust-cachesteps inci.ymlgetsave-ifon the default branch. Pull requests still restore frommain's cache but stop adding PR-scoped entries that eat the ~10 GB quota.Dependabot groups weekly cargo and github-actions bumps so each ecosystem opens one combined PR (
patterns: "*") instead of one per dependency.Warm release cache skips the unsigned Windows release build when
actions/cache/restorereports an exact key hit; restore still runs so the entry is refreshed against idle eviction. Lock or toolchain changes still miss, build, and save as before.Reviewed by Cursor Bugbot for commit e9c66cf. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Trim redundant CI work: group Dependabot updates and limit cache saves
cargoandgithub-actionsin dependabot.yml so matching dependency updates are bundled into single PRsrust-sharedandrust-desktopCI jobs to runs onrefs/heads/main; cache restores are unchangedsteps.cache-restore.outputs.cache-hit == 'true')mainno longer write new Rust cache entries; check thesave-always/cache conditions in ci.ymlMacroscope summarized e9c66cf.
Summary by CodeRabbit