bchanot/claude
1
0
Fork 1
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(make): test target names every red suite and prints a summary

Browse Source 
A full make test printed only the == headers and an aggregate exit code,
so finding the red suite meant re-running every suite one by one (the
pre-merge check of 2026-10-09 took 7.5 min for that reason). The loop now
prints FAIL <suite> as it happens and ends with 'all suites green' or
'<n> suite(s) red: <names>'; the exit code is unchanged.
This commit is contained in:
bchanot
2026-10-09 15:31:26 +02:00
parent ff741e3a82
commit efdd491d63
1 changed files with 7 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Makefile
+7 -4
View File
@@ -34,12 +34,15 @@ test: ## Run deterministic tests hermetically (one: make test suite=lib/tests/x.
@# fire inside the throwaway repos the suites build. The export lives
@# HERE so nobody has to type the (denied) env-prefix form by hand.
@export GIT_CONFIG_GLOBAL=/dev/null GIT_CONFIG_SYSTEM=/dev/null; \
fail=0; for t in $(or $(suite),$(SUITES)); do \
fail=0; red=""; for t in $(or $(suite),$(SUITES)); do \
echo "== $$t"; \
case "$$(basename "$$t")" in \
run-release-candidate.sh) RC_WORK=$$(mktemp -d) RC_TAG=1 bash "$$t" || fail=1 ;; \
*) bash "$$t" || fail=1 ;; \
esac; done; exit $$fail
run-release-candidate.sh) RC_WORK=$$(mktemp -d) RC_TAG=1 bash "$$t" || { fail=1; red="$$red $$t"; echo "FAIL $$t"; } ;; \
*) bash "$$t" || { fail=1; red="$$red $$t"; echo "FAIL $$t"; } ;; \
esac; done; \
if [ $$fail -eq 0 ]; then echo "all suites green"; \
else echo "$$(echo $$red | wc -w | tr -d ' ') suite(s) red:$$red"; fi; \
exit $$fail
scan-secrets: ## Gitleaks sweep: this repo's history + ~/.claude. Extra repos: make scan-secrets repos="path1 path2"
@command -v gitleaks >/dev/null 2>&1 || { echo "gitleaks not installed — https://github.com/gitleaks/gitleaks"; exit 1; }