From 58275ccd52141c3391c99b9decd30196486e4f32 Mon Sep 17 00:00:00 2001 From: Daniel Salazar Date: Sat, 31 Oct 2026 00:42:37 -0700 Subject: [PATCH] ci(puterjs): check out the coverage base leg from this repo (#4431) The base leg checked out the base branch from the PR's head repo. For a fork PR that is the fork's copy of main, which can predate the current Node version and fail npm ci. --- .github/workflows/puterjs-tests.yaml | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/.github/workflows/puterjs-tests.yaml b/.github/workflows/puterjs-tests.yaml index c1a3f0e32..9aa32d4aa 100644 --- a/.github/workflows/puterjs-tests.yaml +++ b/.github/workflows/puterjs-tests.yaml @@ -118,9 +118,13 @@ jobs: fail-fast: false matrix: include: + # The base branch lives in this repo; for a fork PR, the head + # repo's copy of it can be arbitrarily stale. - ref: ${{ github.base_ref }} + repository: ${{ github.repository }} artifact: base - ref: ${{ github.head_ref }} + repository: ${{ github.event.pull_request.head.repo.full_name }} artifact: pr steps: @@ -129,7 +133,7 @@ jobs: uses: actions/checkout@v7 with: ref: ${{ matrix.ref }} - repository: ${{ github.event.pull_request.head.repo.full_name }} + repository: ${{ matrix.repository }} - name: Set up Node.js if: needs.changes.outputs.puterjs == 'true'