Skip to content

Refactor Bazel CI job setup#17704

Merged
bolinfest merged 1 commit intomainfrom
pr17704
Apr 14, 2026
Merged

Refactor Bazel CI job setup#17704
bolinfest merged 1 commit intomainfrom
pr17704

Conversation

@bolinfest
Copy link
Copy Markdown
Collaborator

@bolinfest bolinfest commented Apr 13, 2026

Why

This stack adds a new Bazel CI lane that verifies Rust code behind cfg(not(debug_assertions)), but adding that job directly to .github/workflows/bazel.yml would duplicate the same setup in multiple places. Extracting the shared setup first keeps the follow-up change easier to review and reduces the chance that future Bazel workflow edits drift apart.

What Changed

  • Added .github/actions/prepare-bazel-ci/action.yml as a composite action for the Bazel job bootstrap shared by multiple workflow jobs.
  • Moved the existing Bazel setup, repository-cache restore, and execution-log setup behind that action.
  • Updated the test and clippy jobs in .github/workflows/bazel.yml to call prepare-bazel-ci.
  • Exposed repository-cache-hit and repository-cache-path outputs so callers can keep the existing cache-save behavior without duplicating the restore step.

Verification

  • Parsed .github/workflows/bazel.yml as YAML locally after rebasing the stack.
  • CI will exercise the refactored jobs end to end.

Stack created with Sapling. Best reviewed with ReviewStack.

@bolinfest bolinfest merged commit 440597c into main Apr 14, 2026
47 of 60 checks passed
@bolinfest bolinfest deleted the pr17704 branch April 14, 2026 19:37
@github-actions github-actions bot locked and limited conversation to collaborators Apr 14, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants