Skip to content

fix(ci): skip remote sccache config for fork PRs#388

Merged
drew merged 3 commits intomainfrom
fix-fork-ci-sccache-endpoint
Mar 17, 2026
Merged

fix(ci): skip remote sccache config for fork PRs#388
drew merged 3 commits intomainfrom
fix-fork-ci-sccache-endpoint

Conversation

@drew
Copy link
Collaborator

@drew drew commented Mar 17, 2026

Summary

Avoid exporting SCCACHE_MEMCACHED_ENDPOINT at workflow scope in branch checks so fork PRs do not pass an empty memcached endpoint into sccache.

Related Issue

N/A

Changes

  • Removed the workflow-level SCCACHE_MEMCACHED_ENDPOINT export from branch checks
  • Added a Rust job step that writes SCCACHE_MEMCACHED_ENDPOINT to $GITHUB_ENV only when the repository variable is present
  • Preserved shared remote sccache for internal PRs while allowing fork PRs to run without it

Testing

  • mise run pre-commit passes
  • Unit tests added/updated
  • E2E tests added/updated (if applicable)

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)
  • Architecture docs updated (if applicable)

@drew drew self-assigned this Mar 17, 2026
@drew drew requested review from johntmyers and pimlock March 17, 2026 01:52
@drew drew merged commit bb4545f into main Mar 17, 2026
9 checks passed
@drew drew deleted the fix-fork-ci-sccache-endpoint branch March 17, 2026 02:04
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.

2 participants