From 0eb19fd3c4203ac2801fe6b1a7539942d8baafdd Mon Sep 17 00:00:00 2001 From: Josh Sanchez Date: Wed, 2 Apr 2025 14:54:15 -0500 Subject: [PATCH 1/3] CI: Use pull_request event payload for pull_request_target events --- .github/workflows/build.yml | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 279979bc5..a509232d0 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -32,8 +32,8 @@ jobs: uses: actions/checkout@v4 with: submodules: recursive - ref: ${{ github.event.pull_request_target.head.sha }} - repository: ${{github.event.pull_request_target.head.repo.full_name}} + ref: ${{ github.event.pull_request.head.sha }} + repository: ${{github.event.pull_request.head.repo.full_name}} # Set Git config - name: Git config @@ -96,8 +96,8 @@ jobs: uses: actions/checkout@v4 with: submodules: recursive - ref: ${{ github.event.pull_request_target.head.sha }} - repository: ${{github.event.pull_request_target.head.repo.full_name}} + ref: ${{ github.event.pull_request.head.sha }} + repository: ${{github.event.pull_request.head.repo.full_name}} # Set Git config - name: Git config From 4c8672c9cc0d263b0b2b1aae67dcb171784ad03c Mon Sep 17 00:00:00 2001 From: Josh Sanchez Date: Wed, 2 Apr 2025 15:05:38 -0500 Subject: [PATCH 2/3] CI: Wrap checkout args with strings --- .github/workflows/build.yml | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index a509232d0..496cbc5be 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -24,7 +24,7 @@ jobs: uses: actions/checkout@v4 with: submodules: recursive - ref: ${{ github.event.push.head.sha }} + ref: '${{ github.event.push.after }}' # Checkout the repository (PR from forked repo) - name: Checkout @@ -32,8 +32,8 @@ jobs: uses: actions/checkout@v4 with: submodules: recursive - ref: ${{ github.event.pull_request.head.sha }} - repository: ${{github.event.pull_request.head.repo.full_name}} + ref: '${{ github.event.pull_request.head.sha }}' + repository: '${{github.event.pull_request.head.repo.full_name}}' # Set Git config - name: Git config @@ -88,7 +88,7 @@ jobs: uses: actions/checkout@v4 with: submodules: recursive - ref: ${{ github.event.push.head.sha }} + ref: '${{ github.event.push.after }}' # Checkout the repository (PR from forked repo) - name: Checkout @@ -96,8 +96,8 @@ jobs: uses: actions/checkout@v4 with: submodules: recursive - ref: ${{ github.event.pull_request.head.sha }} - repository: ${{github.event.pull_request.head.repo.full_name}} + ref: '${{ github.event.pull_request.head.sha }}' + repository: '${{github.event.pull_request.head.repo.full_name}}' # Set Git config - name: Git config From a1fd5bae9767519c6bf32ec7d29e43f83f17d3b3 Mon Sep 17 00:00:00 2001 From: Josh Sanchez Date: Thu, 3 Apr 2025 09:56:02 -0500 Subject: [PATCH 3/3] Revert "CI: Wrap checkout args with strings" This reverts commit 4c8672c9cc0d263b0b2b1aae67dcb171784ad03c. --- .github/workflows/build.yml | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 496cbc5be..a509232d0 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -24,7 +24,7 @@ jobs: uses: actions/checkout@v4 with: submodules: recursive - ref: '${{ github.event.push.after }}' + ref: ${{ github.event.push.head.sha }} # Checkout the repository (PR from forked repo) - name: Checkout @@ -32,8 +32,8 @@ jobs: uses: actions/checkout@v4 with: submodules: recursive - ref: '${{ github.event.pull_request.head.sha }}' - repository: '${{github.event.pull_request.head.repo.full_name}}' + ref: ${{ github.event.pull_request.head.sha }} + repository: ${{github.event.pull_request.head.repo.full_name}} # Set Git config - name: Git config @@ -88,7 +88,7 @@ jobs: uses: actions/checkout@v4 with: submodules: recursive - ref: '${{ github.event.push.after }}' + ref: ${{ github.event.push.head.sha }} # Checkout the repository (PR from forked repo) - name: Checkout @@ -96,8 +96,8 @@ jobs: uses: actions/checkout@v4 with: submodules: recursive - ref: '${{ github.event.pull_request.head.sha }}' - repository: '${{github.event.pull_request.head.repo.full_name}}' + ref: ${{ github.event.pull_request.head.sha }} + repository: ${{github.event.pull_request.head.repo.full_name}} # Set Git config - name: Git config