From 05335bbf29b0934004ecb973014fdd2052ad20d4 Mon Sep 17 00:00:00 2001 From: Madelyn Olson Date: Tue, 2 Jun 2026 10:55:34 -0700 Subject: [PATCH] Fix CI to test merge result instead of fork branch in isolation The checkout step was explicitly checking out the PR head ref from the fork repository. This means PRs from stale forks fail if they depend on config changes in main, even though the merge result builds fine. Use refs/pull/N/merge instead, which is GitHub's pre-computed merge of the PR into the base branch. On push/dispatch, check out main directly. Fixes #561 Signed-off-by: Madelyn Olson --- .github/workflows/zola-deploy.yml | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/.github/workflows/zola-deploy.yml b/.github/workflows/zola-deploy.yml index b93b7184..9e39ec3d 100644 --- a/.github/workflows/zola-deploy.yml +++ b/.github/workflows/zola-deploy.yml @@ -21,12 +21,7 @@ jobs: - name: Checkout uses: actions/checkout@v4 with: - ref: >- - ${{ github.event_name == 'pull_request' && - github.head_ref || 'main' }} - repository: >- - ${{ github.event_name == 'pull_request' && - github.event.pull_request.head.repo.full_name || 'valkey-io/valkey-io.github.io' }} + ref: ${{ github.event_name != 'pull_request' && 'main' || '' }} path: website - name: Checkout valkey-doc