Skip to content

chore(conformance): remove the sendspin-jvm client adapter - #162

Merged
chrisuthe merged 2 commits into
Sendspin:mainfrom
chrisuthe:chrisuthe/task/remove-the-sendspin-jvm-client-adapter
Oct 14, 2026
Merged

chrisuthe merged 2 commits into
Sendspin:mainfrom
chrisuthe:chrisuthe/task/remove-the-sendspin-jvm-client-adapter

Conversation

@chrisuthe

Copy link
Copy Markdown
Member

Removes the sendspin-jvm (Kotlin/Gradle) client adapter from the harness, along with the Java-only pieces that existed solely for it.

  • Delete adapters/sendspin-jvm/ and src/conformance/adapters/sendspin_jvm_client.py
  • Drop the sendspin-jvm entry from IMPLEMENTATIONS (which also stops setup_repositories.py cloning it)
  • Drop _gradle_build_result and its _build_plan() step; remove find_java() from toolchains.py
  • CI: remove the Setup Java step and sendspin-jvm from the published --to matrix
  • README: remove the sendspin-jvm coverage bullet; reword the test_undeclared_formats.py module docstring so it doesn't name a removed implementation

Regression detection only compares cases present in the current run, so the dropped jvm cases won't be flagged as regressions.

Verified: compileall, unittest discover -s tests (16 OK), clean imports, run_all.py --help, git diff --check, and git grep -i -E 'jvm|java|gradle|kotlin' returns nothing.

Drop the Kotlin/Gradle adapter, its Python launcher, registry entry and
build step, along with find_java() and the CI Java setup that existed
only for it.
…ve-the-sendspin-jvm-client-adapter

# Conflicts:
#	.github/workflows/publish.yml
#	README.md
#	adapters/sendspin-jvm/client/src/main/kotlin/com/sendspin/conformance/Main.kt
#	src/conformance/build.py
#	src/conformance/implementations.py
#	tests/test_undeclared_formats.py
@chrisuthe
chrisuthe marked this pull request as ready for review October 14, 2026 17:54
@chrisuthe
chrisuthe merged commit 4cae04b into Sendspin:main Oct 14, 2026
1 of 2 checks passed
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.

1 participant