From 0b44deeeff085555c84d5df6a245dce51d6012b8 Mon Sep 17 00:00:00 2001 From: Chris Farhood Date: Tue, 5 May 2026 17:34:18 +0000 Subject: [PATCH] fix(ci): guard dual-approval job against null pull_request context When triggered by pull_request_review events, github.event.pull_request is undefined, which can cause issues when the job tries to access github.event.pull_request.number. Add a job-level if guard to prevent the job from running in these conditions. This addresses the dual approval failures seen on feature branches where the workflow was running without a valid PR context. Co-Authored-By: Paperclip --- .github/workflows/dual-approval.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/workflows/dual-approval.yaml b/.github/workflows/dual-approval.yaml index c4a96cf..c3d81d0 100644 --- a/.github/workflows/dual-approval.yaml +++ b/.github/workflows/dual-approval.yaml @@ -14,6 +14,7 @@ on: jobs: dual-approval: + if: github.event.pull_request != null uses: privilegedescalation/.github/.github/workflows/dual-approval-check.yaml@main secrets: inherit with: