Skip to content

Use GHA id-token for sccache-dist auth token#44

Merged
rapids-bot[bot] merged 1 commit intorapidsai:mainfrom
trxcllnt:fea/sccache-dist-id-token
Feb 10, 2026
Merged

Use GHA id-token for sccache-dist auth token#44
rapids-bot[bot] merged 1 commit intorapidsai:mainfrom
trxcllnt:fea/sccache-dist-id-token

Conversation

@trxcllnt
Copy link
Copy Markdown
Contributor

Use the GitHub OIDC token for sccache-dist auth instead of the org's secrets.GIST_REPO_READ_ORG_GITHUB_TOKEN personal access token.

This change is already implemented, this PR just removes any references to GIST_REPO_READ_ORG_GITHUB_TOKEN.

@trxcllnt trxcllnt requested a review from a team as a code owner February 10, 2026 22:23
@trxcllnt trxcllnt added improvement Improves an existing functionality non-breaking Introduces a non-breaking change labels Feb 10, 2026
@trxcllnt trxcllnt requested a review from msarahan February 10, 2026 22:23
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Feb 10, 2026

📝 Walkthrough

Walkthrough

Removed sccache-dist-token-secret-name: GIST_REPO_READ_ORG_GITHUB_TOKEN property from workflow job definitions across three GitHub Actions workflow files. The change affects 15 workflow step configurations in total with no modifications to control flow or error handling.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow Configuration
.github/workflows/build.yaml, .github/workflows/pr.yaml, .github/workflows/test.yaml
Removed sccache-dist-token-secret-name: GIST_REPO_READ_ORG_GITHUB_TOKEN property from workflow job definitions (4 jobs in build.yaml, 8 in pr.yaml, and 3 in test.yaml).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes


Note

🎁 Summarized by CodeRabbit Free

Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login.

Comment @coderabbitai help to get the list of available commands and usage tips.

@jameslamb jameslamb removed the request for review from msarahan February 10, 2026 23:10
@trxcllnt
Copy link
Copy Markdown
Contributor Author

/merge

@rapids-bot rapids-bot Bot merged commit 90049ff into rapidsai:main Feb 10, 2026
74 checks passed
@trxcllnt trxcllnt deleted the fea/sccache-dist-id-token branch February 10, 2026 23:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improves an existing functionality non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants