From 4a2172865ed53765d3e7d8591582f6c305c0e221 Mon Sep 17 00:00:00 2001 From: Michal Harakal Date: Wed, 26 Aug 2026 17:49:05 +0200 Subject: [PATCH] ci: give apiCheck its own named PR leg MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit apiCheck already ran in PR CI — inside the golden-parity matrix leg (#1005) — and it caught both recent incidents; but a stale api dump surfaced as a red 'test (golden-parity)' that nobody associates with binary compatibility, and #1171 merged through exactly that red, breaking apiCheck on develop until #1174. The check moves to its own matrix entry: a stale dump now reads as 'test (api-compatibility) ✘' with the fix ('run apiDump, commit the dumps') one association away. golden-parity goes back to being purely the packed-encoding gate. The build-job aggregator needs the whole matrix, so no other wiring changes. Co-Authored-By: Claude Fable 5 --- .github/workflows/build.yml | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index b03a5f1a0..923e3ae43 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -54,13 +54,17 @@ jobs: tasks: testAndroidHostTest # golden-parity: the SKEEP-003 packed-encoding gate (#1005). Bit-identical decode / # scalar-kernel / TurboQuant digests on JVM and Kotlin/Native plus the dispatch parity - # tests, and the binary-compatibility check (apiCheck) that the contributing docs require - # but the per-target test legs do not run. Fast (a few minutes) so it fails early. + # tests. Fast (a few minutes) so it fails early. - name: golden-parity tasks: >- - apiCheck :skainet-backends:skainet-backend-cpu:jvmTest --tests sk.ainet.exec.golden.* :skainet-backends:skainet-backend-cpu:linuxX64Test --tests sk.ainet.exec.golden.* + # api-compatibility: the binary-compatibility check the contributing docs require but the + # per-target test legs do not run. Its own leg — not folded into golden-parity — so a + # stale api dump reads as "api-compatibility ✘" instead of a red parity leg nobody + # associates with dumps (#1171 merged a stale dump exactly that way; #1174 repaired it). + - name: api-compatibility + tasks: apiCheck permissions: contents: read