Skip to content

Fix/ci release docs deploy#5881

Merged
jyotshnayaparla-00 merged 1 commit intomainfrom
fix/ci-release-docs-deploy
Mar 20, 2026
Merged

Fix/ci release docs deploy#5881
jyotshnayaparla-00 merged 1 commit intomainfrom
fix/ci-release-docs-deploy

Conversation

@jyotshnayaparla-00
Copy link
Contributor

add persist-credentials: false to the checkout step so no credentials are stored, and only the deploy action's own app token is used.

@jyotshnayaparla-00 jyotshnayaparla-00 requested a review from gz March 20, 2026 16:21
actions/checkout@v6 stores credentials using includeIf scoped to the repo
and its worktrees. The docs deploy action creates a temporary worktree,
causing checkout's GITHUB_TOKEN credentials to override the app token,
resulting in a 403 as github-actions[bot].

Setting persist-credentials: false prevents checkout from storing any
credentials, so only the deploy action's own app token is used.
@jyotshnayaparla-00 jyotshnayaparla-00 force-pushed the fix/ci-release-docs-deploy branch from 780a503 to b43d0f7 Compare March 20, 2026 16:21
Copy link
Contributor

@gz gz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this seems ot be the same issue we're running into

JamesIves/github-pages-deploy-action#1928

@jyotshnayaparla-00 jyotshnayaparla-00 added this pull request to the merge queue Mar 20, 2026
Copy link

@mythical-fred mythical-fred left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Merged via the queue into main with commit 2dc83b9 Mar 20, 2026
1 check passed
@jyotshnayaparla-00 jyotshnayaparla-00 deleted the fix/ci-release-docs-deploy branch March 20, 2026 17:53
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.

3 participants