From 2d282208690f2a10d9f387d6d5d4f88eb907d2c0 Mon Sep 17 00:00:00 2001 From: Lilleman auf Larv Date: Thu, 3 Sep 2026 22:36:48 +0200 Subject: [PATCH] Diff the whole push range and de-duplicate the gate-skip note --- .github/workflows/test.yml | 2 +- README.md | 9 ++++----- 2 files changed, 5 insertions(+), 6 deletions(-) diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 03ccd42..d6336ba 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -24,7 +24,7 @@ jobs: git fetch --no-tags origin "$GITHUB_BASE_REF" base="origin/$GITHUB_BASE_REF" else - base="HEAD^" + base='${{ github.event.before }}' fi if git diff --quiet "$base"...HEAD -- '*.go' go.mod go.sum Dockerfile .dockerignore .github data README.md; then echo "code=false" >> "$GITHUB_OUTPUT" diff --git a/README.md b/README.md index 727c6c3..e0ccb1b 100644 --- a/README.md +++ b/README.md @@ -491,11 +491,10 @@ docker compose run --rm --user "$(id -u):$(id -g)" tidy # go mod tidy ``` Every pull request runs `docker build .` against both the latest and the lowest -supported Go, and must pass before it can be merged. That build is the whole -gate — vet, complexity, format check and tests — so run it locally before pushing. -A PR that touches none of the Go and module files, the Dockerfile, -`.dockerignore`, the workflow, the shipped data nor the README skips the build -and passes as-is (see [Decisions](#decisions)): +supported Go, and must pass before it can be merged — unless it touches none of +the files the build and its tests read, in which case it's skipped (see +[Decisions](#decisions)). That build is the whole gate — vet, complexity, format +check and tests — so run it locally before pushing: ```sh docker build . # latest