Skip to content

Kotlin's jvmTarget 17, as the project and Paxtools 6 are - #88

Merged
adamjohnwright merged 1 commit into
mainfrom
build/jvm-target-17
Oct 29, 2026
Merged

adamjohnwright merged 1 commit into
mainfrom
build/jvm-target-17

Conversation

@adamjohnwright

Copy link
Copy Markdown
Contributor

Small follow-ups from the spec 001 reviews.

  • jvmTarget: Kotlin compiled for Java 1.8, while the project's jdk.version, its Docker images and Paxtools 6 are all Java 17. The classes are now version 61. That also brings JUnit 6's inline assertThrows into reach, and the tests now use it (it wouldn't compile before).
  • Output: the fixture's output is unchanged to the byte, and 93 tests pass.
  • differences.md: says that an error reading an event's id is reported as itself, while one reading a species' id ends the run with its stack trace (from From #86's review: the Jenkinsfile's credentials quoted, what a bad id writes #87's review).

🤖 Generated with Claude Code

Kotlin compiled for Java 1.8 while the project's jdk.version, its images
and Paxtools 6 are Java 17, which also kept JUnit 6's inline assertThrows
out of reach; the tests now use it. Output is unchanged. differences.md
says that an error reading an event's id is reported as itself, while one
reading a species' id ends the run with its stack trace.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@adamjohnwright
adamjohnwright merged commit d30f09c into main Oct 29, 2026
6 checks passed
@adamjohnwright
adamjohnwright deleted the build/jvm-target-17 branch October 29, 2026 14:37
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