Skip to content

chore(bcv): refresh stale skainet-compile-hlo API dump (pre-existing apiCheck failure on develop) - #1047

Merged
michalharakal merged 1 commit into
developfrom
fix/bcv-refresh-compile-hlo-api-dump
Aug 22, 2026
Merged

michalharakal merged 1 commit into
developfrom
fix/bcv-refresh-compile-hlo-api-dump

Conversation

@michalharakal

Copy link
Copy Markdown
Contributor

Summary

./gradlew apiCheck fails on develop for :skainet-compile:skainet-compile-hlo: TypeMapper.DYNAMIC_DIM, TypeMapper.formatShape(...) and the TypeMapper.Companion were added without regenerating skainet-compile/skainet-compile-hlo/api/jvm/skainet-compile-hlo.api (CI runs per-target *Test legs and never apiCheck, so it went unnoticed). Regenerated with ./gradlew apiDump — additions only, no source change.

Split out of #1045 so that the memory-architecture slices (#932 → #1001) can keep apiCheck in their local PR gate; #1045 will be rebased onto develop once this is merged.

Test plan

Full local gate (JDK 25): jvmTest ✔ (3m 53s) · apiCheck ✔ · verifyNpmPins jsTest wasmJsTest wasmWasiTest ✔ (2m 25s) · linuxX64Test ✔ · assemble ✔ · :skainet-test:skainet-test-java:test ✔.

🤖 Generated with Claude Code

…drift on develop)

`./gradlew apiCheck` fails on develop for :skainet-compile:skainet-compile-hlo
(TypeMapper.DYNAMIC_DIM, formatShape, Companion were added without
regenerating api/jvm/skainet-compile-hlo.api; CI does not run apiCheck).
Regenerated with `./gradlew apiDump` — additions only — so the local PR
gate (scripts/pr-gate.sh) is green for the memory-architecture slices.
Unrelated to the dtype bridge itself.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@michalharakal
michalharakal requested a review from aharakal August 22, 2026 16:10
@michalharakal
michalharakal merged commit c7d0c4f into develop Aug 22, 2026
11 checks passed
@michalharakal
michalharakal deleted the fix/bcv-refresh-compile-hlo-api-dump branch August 22, 2026 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants