ci: try cached workflows - #816
Conversation
There was a problem hiding this comment.
🟡 Changes recommended
Cache-warming runs use an empty workspace, race across matrix jobs, and some cleanup permissions do not reach called workflows.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
Updates CI workflows to trial the new Bazel cache setup using intentionally unpinned @cache workflow branches.
Changes:
- Switches shared PR and maintenance workflows to cached variants.
- Adds cache-cleanup permissions.
- Adds scheduled and default-branch downstream cache warming.
File summaries
| File | Description |
|---|---|
.github/workflows/on-pr.yml |
Uses cached PR and documentation workflows. |
.github/workflows/downstream_compatibility.yml |
Adds cache-warming triggers and permissions. |
.github/workflows/daily.yml |
Uses cached maintenance workflow. |
Review details
- Files reviewed: 3/3 changed files
- Comments generated: 3
- Review effort level: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
Documentation preview for this pull request is available at: |
There was a problem hiding this comment.
🟡 Changes recommended
Pull-request jobs unnecessarily receive cache-deletion permissions.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Review details
- Files reviewed: 3/3 changed files
- Comments generated: 2
- Review effort level: Balanced
try using the workflows based on the new setup-bazel-cache action
They are intentionally unpinned