Skip to content

fix(ci): skip GHCR publish tests on dependabot PRs - #293

Merged
marcusburghardt merged 1 commit into
complytime:mainfrom
marcusburghardt:fix/skip-ghcr-test-dependabot
May 29, 2026
Merged

fix(ci): skip GHCR publish tests on dependabot PRs#293
marcusburghardt merged 1 commit into
complytime:mainfrom
marcusburghardt:fix/skip-ghcr-test-dependabot

Conversation

@marcusburghardt

@marcusburghardt marcusburghardt commented May 29, 2026

Copy link
Copy Markdown
Member

Summary

dependabot[bot] is not an org member, so the org membership check in reusable_publish_ghcr.yml always fails for dependabot PRs that touch GHCR-related files (e.g., bumping docker/setup-qemu-action). The fork detection guard from #291 does not help because dependabot branches live in the same repository, not a fork.

This adds a github.actor != 'dependabot[bot]' condition to both GHCR test jobs (test-unprotected-org-member and test-attestations-override) so they skip cleanly instead of producing false failures.

Related Issues

Review Hints

dependabot[bot] is not an org member, so the org membership check
in reusable_publish_ghcr.yml always fails for dependabot PRs that
touch GHCR-related files. The fork detection guard from complytime#291 does
not help because dependabot branches live in the same repository.

Add an actor exclusion so the GHCR integration test jobs skip
cleanly for dependabot PRs instead of producing false failures.

Assisted-by: OpenCode (claude-opus-4-6)
Signed-off-by: Marcus Burghardt <maburgha@redhat.com>
@marcusburghardt
marcusburghardt merged commit d35e6fc into complytime:main May 29, 2026
12 checks passed
@marcusburghardt
marcusburghardt deleted the fix/skip-ghcr-test-dependabot branch May 29, 2026 13:19
@beatrizmcouto beatrizmcouto moved this to Backlog in ComplyTime planning Jun 8, 2026
@github-project-automation github-project-automation Bot moved this from Backlog to Done ✔️ in ComplyTime planning Jun 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done ✔️

Development

Successfully merging this pull request may close these issues.

3 participants