timothy/ersatztv
1
0
Fork 0
Code Issues 51 Pull Requests 5 Actions Packages Projects Releases 9 Wiki Activity

ci(416): skip heavy jobs on docs-only changes without bricking the merge gate #422

Merged
timothy merged 3 commits from ci/416-docs-only-skip into main 2026-07-17 20:10:25 +02:00
Conversation 3 Commits 3 Files Changed 4
+262 -2
3 Commits
Author SHA1 Message Date
timothy f7b97adce8 ci(416): harden docs-only detection with --no-renames (review finding)
Build ErsatzTV Image / CI image pin matches docker/ci (pull_request) Successful in 5s
Details
Build ErsatzTV Image / Docs update reminder (pull_request) Successful in 7s
Details
Build ErsatzTV Image / decisions.md append-only (pull_request) Successful in 5s
Details
Build ErsatzTV Image / Functional E2E (curl contracts) (pull_request) Successful in 5m6s
Details
Build ErsatzTV Image / API docs in sync (OpenAPI + endpoint index) (pull_request) Successful in 8s
Details
Build ErsatzTV Image / Formatting (changed .cs conform to .editorconfig) (pull_request) Successful in 5s
Details
Build ErsatzTV Image / EF migration integrity (SQLite + MySql) (pull_request) Successful in 15m9s
Details
Build ErsatzTV Image / Build & test (.NET) (pull_request) Successful in 18m34s
Details
Build ErsatzTV Image / Build & push image (amd64) (pull_request) Has been skipped
Details 
The changed-set git diff had rename detection on by default, so a code->docs
rename (Foo.cs -> docs/Foo.md) showed only the destination and was misclassified
as docs-only, skipping required tests on a code change. --no-renames surfaces the
source deletion -> full matrix. Empirically verified. This is the cold-review
MEDIUM; it was applied in the working tree but never committed before the first
push (index/worktree mismatch) — committing it now.

Refs #416
 2026-07-17 19:44:25 +02:00
timothy 92bb63b6da docs(416): document docs-only CI skip 
ci-cd.md gains a 'Docs-only skip' section + triggers-table note; decisions.md
records the decision. Cross-refs the separate PR-vs-main rerun redundancy (#420)
and the within-run triple build (#398).

Refs #416
 2026-07-17 19:44:25 +02:00
timothy 5ba737bec7 ci(416): skip heavy jobs on docs-only changes 
Docs-only changes (docs/** or *.md) ran the full docker-build matrix (~9 min).
Each heavy job (test, migrations, functional-e2e, build) now runs
scripts/ci-detect-docs-only.sh as its first post-checkout step and gates every
real step on docs_only!='true'. The jobs still RUN and report success in
seconds, so the two required contexts keep reporting — a docs-only PR stays
mergeable (never an if:-skipped required job; Gitea 1.25.4 reports if-skip as
'skipped', verified with a throwaway probe PR). build skips its image steps on a
docs-only push to main; tag builds force docs_only=false. Detection uses
--no-renames so a code->docs rename can never be misclassified as docs-only.

Refs #416
 2026-07-17 19:44:25 +02:00