Skip to content

ci(codegen): compile java and swift clients against published SDKs - #366

Merged
scarmuega merged 1 commit into
mainfrom
ci/codegen-check-published-sdks
Sep 28, 2026
Merged

scarmuega merged 1 commit into
mainfrom
ci/codegen-check-published-sdks

Conversation

@scarmuega

Copy link
Copy Markdown
Contributor

Why

The java-client and swift-client compile checks built against pinned SDK commits (java-sdk d7a6dc4 as 0.15.0-SNAPSHOT; swift-sdk 02fa0f2 through a locally tagged SwiftPM mirror) because the SDKs had no release. Both are now published:

Change

Remove both stand-in blocks and the -Dtx3.sdk.version / mirror overrides. The rendered projects now build exactly as generated: Java resolves the template's tx3.sdk.version (0.15.0) from Central using the runner's Maven (the generated project has no wrapper; ubuntu-latest ships Maven), and Swift resolves from: "0.15.0" from the public tag. The script header's "compiles against the published SDK release" is accurate again.

Verification

  • Local (macOS, Swift 6.2.3, tx3c 0.25.0 built from this branch's base): codegen-compile-check.sh swift-client resolved swift-sdk at 0.15.0 and built both transfer and complex fixtures.
  • java-client: no local JDK; relies on this PR's codegen compile (java-client) job.
  • bash -n passes.

🤖 Generated with Claude Code

land.tx3:tx3-sdk:0.15.0 is on Maven Central and swift-sdk v0.15.0 is
tagged, so drop the pinned-commit stand-ins: the generated Maven project
resolves 0.15.0 from Central with the runner's Maven, and the generated
SwiftPM package resolves the published tag directly.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@scarmuega
scarmuega merged commit dabcab4 into main Sep 28, 2026
16 checks passed
@scarmuega
scarmuega deleted the ci/codegen-check-published-sdks branch September 28, 2026 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant