larvit/fejkdata
3
0
Fork 0
Code Pull Requests 1 Actions Activity

Run the merge gate only for changes its tests read #7

Merged
lilleman merged 4 commits from gate-skip-untested-changes into main 2026-09-03 23:06:33 +02:00
Conversation 0 Commits 4 Files Changed 2
+5 -6
2 changed files with 5 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 2d28220869 - Show all commits
.github/workflows/test.yml
+1 -1
View File
@@ -24,7 +24,7 @@ jobs:
git fetch --no-tags origin "$GITHUB_BASE_REF" git fetch --no-tags origin "$GITHUB_BASE_REF"
base="origin/$GITHUB_BASE_REF" base="origin/$GITHUB_BASE_REF"
else else
base="HEAD^" base='${{ github.event.before }}'
fi fi
if git diff --quiet "$base"...HEAD -- '*.go' go.mod go.sum Dockerfile .dockerignore .github data README.md; then if git diff --quiet "$base"...HEAD -- '*.go' go.mod go.sum Dockerfile .dockerignore .github data README.md; then
echo "code=false" >> "$GITHUB_OUTPUT" echo "code=false" >> "$GITHUB_OUTPUT"
README.md
+4 -5
View File
@@ -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 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 supported Go, and must pass before it can be merged — unless it touches none of
gate — vet, complexity, format check and tests — so run it locally before pushing. the files the build and its tests read, in which case it's skipped (see
A PR that touches none of the Go and module files, the Dockerfile, [Decisions](#decisions)). That build is the whole gate — vet, complexity, format
`.dockerignore`, the workflow, the shipped data nor the README skips the build check and tests — so run it locally before pushing:
and passes as-is (see [Decisions](#decisions)):
```sh ```sh
docker build . # latest docker build . # latest