Skip to content

ci: make sccache optional in Rust validation - #77

Merged
proerror77 merged 1 commit into
mainfrom
codex/ci-sccache-optional
Jul 17, 2026
Merged

ci: make sccache optional in Rust validation#77
proerror77 merged 1 commit into
mainfrom
codex/ci-sccache-optional

Conversation

@proerror77

Copy link
Copy Markdown
Owner

Change contract

Keep Rust Workspace and Rust HFT Engine Fast Lane validating when the optional sccache action cannot reach GitHub REST APIs.

What changed

  • Removed mozilla-actions/sccache-action and its wrapper/env wiring from the two affected CI jobs.
  • Kept Swatinem/rust-cache and every existing Cargo validation command.

Acceptance evidence

  • git diff --check passed.
  • Workflow YAML parsed successfully.
  • Static scan confirms no remaining sccache action, wrapper, env flag, or action-output cache-key reference in this workflow.
  • Focused GitHub Actions validation is pending on this PR.

Out of scope

Dependency and merge order

Base: main. Merge this PR before rerunning PR #67, so its Rust CI no longer depends on the failing optional action.

Rollout and rollback

CI-only change. Revert this one commit to restore sccache; validation remains functionally equivalent but builds may run slower.

@coderabbitai

coderabbitai Bot commented Jul 16, 2026

Copy link
Copy Markdown

Caution

Review failed

An error occurred during the review process. Please try again later.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/ci-sccache-optional

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.

@proerror77
proerror77 merged commit 56e86fd into main Jul 17, 2026
17 checks passed
@proerror77
proerror77 deleted the codex/ci-sccache-optional branch July 17, 2026 00:07
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.

1 participant