Skip to content

fix(ci): handle fork PRs in GitHub Actions workflows#499

Merged
alirezarezvani merged 4 commits intodevfrom
claude/audit-pr-498-workflow-Pa5Ku
Apr 8, 2026
Merged

fix(ci): handle fork PRs in GitHub Actions workflows#499
alirezarezvani merged 4 commits intodevfrom
claude/audit-pr-498-workflow-Pa5Ku

Conversation

@alirezarezvani
Copy link
Copy Markdown
Owner

Fork-based PRs (like PR #498) caused all CI checks to fail due to:

  • ci-quality-gate: checkout failed because fork branch names don't exist
    in the base repo. Now uses commit SHA for PR events.
  • skill-security-audit: comment posting failed with read-only GITHUB_TOKEN.
    Now continues on error and writes results to job summary as fallback.
  • claude-code-review: fallback comment step failed silently. Now continues
    on error and writes status to job summary.

https://claude.ai/code/session_01X1RKFAkEwxgg6gQvJG1KCa

alirezarezvani and others added 4 commits April 4, 2026 02:22
Release: dev → main — 28 new skills, 7 community PRs, full sync
Fork-based PRs (like PR #498) caused all CI checks to fail due to:
- ci-quality-gate: checkout failed because fork branch names don't exist
  in the base repo. Now uses commit SHA for PR events.
- skill-security-audit: comment posting failed with read-only GITHUB_TOKEN.
  Now continues on error and writes results to job summary as fallback.
- claude-code-review: fallback comment step failed silently. Now continues
  on error and writes status to job summary.

https://claude.ai/code/session_01X1RKFAkEwxgg6gQvJG1KCa
@alirezarezvani alirezarezvani changed the base branch from main to dev April 8, 2026 22:14
@alirezarezvani alirezarezvani merged commit 44654d2 into dev Apr 8, 2026
3 of 4 checks passed
@alirezarezvani alirezarezvani deleted the claude/audit-pr-498-workflow-Pa5Ku branch April 8, 2026 22:15
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