From 9edaa85e44c51c621fb9b35b056b65fcd906dc23 Mon Sep 17 00:00:00 2001 From: Max Ostapenko <1611259+max-ostapenko@users.noreply.github.com> Date: Tue, 22 Sep 2026 11:13:16 +0200 Subject: [PATCH 1/2] ci: update PR checkout logic and conditional comment in wpt-test workflow Signed-off-by: Max Ostapenko <1611259+max-ostapenko@users.noreply.github.com> --- .github/workflows/wpt-test.yml | 15 +++++++++++++-- 1 file changed, 13 insertions(+), 2 deletions(-) diff --git a/.github/workflows/wpt-test.yml b/.github/workflows/wpt-test.yml index fabc712..6e152ed 100644 --- a/.github/workflows/wpt-test.yml +++ b/.github/workflows/wpt-test.yml @@ -19,9 +19,20 @@ jobs: - name: Checkout uses: actions/checkout@v7 with: - ref: ${{ github.event.pull_request.head.sha }} fetch-depth: 0 + - name: "Fetch PR changes (Internal branch: full checkout)" + if: github.event_name == 'pull_request_target' && github.event.pull_request.head.repo.full_name == github.repository + run: | + git fetch origin ${{ github.event.pull_request.head.sha }} + git checkout ${{ github.event.pull_request.head.sha }} + + - name: "Fetch PR changes (Fork PR: dist/ only)" + if: github.event_name == 'pull_request_target' && github.event.pull_request.head.repo.full_name != github.repository + run: | + git fetch origin ${{ github.event.pull_request.head.sha }} + git checkout ${{ github.event.pull_request.head.sha }} -- dist/ + - name: Install dependencies run: npm install jest webpagetest @@ -40,7 +51,7 @@ jobs: - name: Add comment to PR uses: mshick/add-pr-comment@v3 - if: always() + if: always() && hashFiles('comment.md') != '' with: refresh-message-position: true message-path: comment.md From 22ac0279a345f920ccfac3e9a45d2b5fe7dd9d53 Mon Sep 17 00:00:00 2001 From: Max Ostapenko <1611259+max-ostapenko@users.noreply.github.com> Date: Tue, 22 Sep 2026 11:21:36 +0200 Subject: [PATCH 2/2] ci: wrap long if conditions in wpt-test to satisfy yamllint line-length Signed-off-by: Max Ostapenko <1611259+max-ostapenko@users.noreply.github.com> --- .github/workflows/wpt-test.yml | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/.github/workflows/wpt-test.yml b/.github/workflows/wpt-test.yml index 6e152ed..78a954a 100644 --- a/.github/workflows/wpt-test.yml +++ b/.github/workflows/wpt-test.yml @@ -22,13 +22,17 @@ jobs: fetch-depth: 0 - name: "Fetch PR changes (Internal branch: full checkout)" - if: github.event_name == 'pull_request_target' && github.event.pull_request.head.repo.full_name == github.repository + if: | + github.event_name == 'pull_request_target' && + github.event.pull_request.head.repo.full_name == github.repository run: | git fetch origin ${{ github.event.pull_request.head.sha }} git checkout ${{ github.event.pull_request.head.sha }} - name: "Fetch PR changes (Fork PR: dist/ only)" - if: github.event_name == 'pull_request_target' && github.event.pull_request.head.repo.full_name != github.repository + if: | + github.event_name == 'pull_request_target' && + github.event.pull_request.head.repo.full_name != github.repository run: | git fetch origin ${{ github.event.pull_request.head.sha }} git checkout ${{ github.event.pull_request.head.sha }} -- dist/