Skip to content

ci: treat dependabot PRs like fork PRs in pre-merge queue#5810

Merged
jyotshnayaparla-00 merged 1 commit intomainfrom
fix-dependabot-premergequeue
Mar 12, 2026
Merged

ci: treat dependabot PRs like fork PRs in pre-merge queue#5810
jyotshnayaparla-00 merged 1 commit intomainfrom
fix-dependabot-premergequeue

Conversation

@jyotshnayaparla-00
Copy link
Contributor

Dependabot creates branches in the main repo (not a fork), so the existing fork check passes , but GitHub withholds repository secrets from Dependabot PRs, leaving CI_ACCESS_APP_PKEY empty and causing create-github-app-token to fail.

Add github.actor != 'dependabot[bot]' guards alongside the fork checks so dependabot PRs skip secret-dependent steps (app token, sccache, auto-commit) and use the plain checkout path instead.

Fixes #5746

Dependabot creates branches in the main repo (not a fork), so the
existing fork check passes — but GitHub withholds repository secrets
from Dependabot PRs, leaving CI_ACCESS_APP_PKEY empty and causing
create-github-app-token to fail.

Add `github.actor != 'dependabot[bot]'` guards alongside the fork
checks so dependabot PRs skip secret-dependent steps (app token,
sccache, auto-commit) and use the plain checkout path instead.

Fixes #5746
@jyotshnayaparla-00 jyotshnayaparla-00 self-assigned this Mar 12, 2026
@jyotshnayaparla-00 jyotshnayaparla-00 requested a review from gz March 12, 2026 16:27
@jyotshnayaparla-00 jyotshnayaparla-00 added this pull request to the merge queue Mar 12, 2026
@jyotshnayaparla-00 jyotshnayaparla-00 removed this pull request from the merge queue due to a manual request Mar 12, 2026
@jyotshnayaparla-00 jyotshnayaparla-00 added this pull request to the merge queue Mar 12, 2026
Copy link
Collaborator

@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 ab6c6ae Mar 12, 2026
1 check passed
@jyotshnayaparla-00 jyotshnayaparla-00 deleted the fix-dependabot-premergequeue branch March 12, 2026 20:03
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.

dependabot PR's don't pass pre-merge queue

3 participants