Skip to content

Commit 1c44499

Browse files
Copilotgounthar
andcommitted
Add comment documenting IS_FORK bug fix
Document the fix that prevents IS_FORK from being incorrectly set to true for push events by checking event type before accessing pull_request fields. This addresses the issue where workflow run #20105424925 skipped Docker builds. Co-authored-by: gounthar <116569+gounthar@users.noreply.github.com>
1 parent 81575ed commit 1c44499

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

.github/workflows/github-docker-registry-push.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,8 @@ jobs:
2222
steps:
2323
- name: Check if PR is from a fork
2424
# This step checks if the pull request is from a fork
25+
# Fixed: Check event type before accessing pull_request fields to prevent
26+
# IS_FORK being incorrectly set to true for push events (issue #20105424925)
2527
run: |
2628
if [ "${{ github.event_name }}" = "pull_request" ]; then
2729
if [ '${{ github.event.pull_request.head.repo.full_name }}' != '${{ github.repository }}' ]; then

0 commit comments

Comments
 (0)