Repository navigation
chore: upgrade Gradle wrapper to 9.8.2 - #439
Draft
katerina20 wants to merge 3 commits into
Draft
katerina20 wants to merge 3 commits into
katerina20 wants to merge 3 commits into
Conversation
Bumps [gradle-wrapper](https://github.com/gradle/gradle) from 7.6.4 to 9.8.2. - [Release notes](https://github.com/gradle/gradle/releases) - [Commits](gradle/gradle@v7.6.4...v9.8.2) --- updated-dependencies: - dependency-name: gradle-wrapper dependency-version: 9.8.2 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
Gradle 9 requires JDK 17+ to run and removes several APIs this build used. - CI, docs, dependency submission and JitPack now run Gradle on JDK 17 - library still compiles with --release 8, so the published jar keeps Java 8 bytecode and API - javadoc runs with -source 8 so lombok.var is accepted on JDK 17 - migrate project-level sourceCompatibility and JaCoCo report settings to their Gradle 9 replacements - bump JaCoCo 0.8.5 -> 0.8.16 (0.8.5 cannot instrument Java 17 runtimes) Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Gradle and the unit tests now run on JDK 17, so nothing in CI ran the library on Java 8 anymore. java8SmokeTest runs the built jar on a Java 8 toolchain against a local stub server: it constructs a Client, performs a real HTTP round trip, deserializes a polymorphic Project with an enum, and parses an error response. It catches Java 9+ API calls or dependencies that only fail at runtime on Java 8. CI installs JDK 8 next to JDK 17 for this step only; Gradle still runs on 17. The task is not part of `check`, so local builds do not need a JDK 8. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
katerina20
marked this pull request as draft
November 13, 2026 07:14
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Replaces Dependabot PR #428, which fails because Gradle 9 needs JDK 17+ to run and drops APIs this build used.
Gradle now runs on JDK 17. The published library still targets Java 8.
Changes
jitpack.ymlpins JitPack to JDK 17. JitPack already used 17, so this only makes it explicit.options.release = 8, so it's checked against the Java 8 API, not just built as Java 8 bytecode. Javadoc runs with-source 8so JDK 17's javadoc acceptslombok.var.sourceCompatibilityand the old JaCoCo report settings, which Gradle 9 removed.java8SmokeTesttask runs the built jar on a real Java 8 runtime against a local stub server: a real HTTP round trip, project deserialization and error parsing. CI installs JDK 8 for this step only. It isn't part ofbuild, so local builds don't need a JDK 8.AGENTS.md: updated for the new JDK requirement and the smoke test command.The published
.modulestill declaresorg.gradle.jvm.version: 8with the same dependencies as 1.38.0. No production code changed.Testing (locally, Gradle 9.8.2 on JDK 17)
./gradlew clean build,test,jacocoTestReportandjavadocpass. All 892 tests pass../gradlew java8SmokeTestpasses on Java 1.8.0_392. I checked that it catches a regression: withrelease = 8removed, aList.of()call compiles fine on JDK 17 but makes the smoke test fail on Java 8 withNoSuchMethodError.Notes
version '1.38.0'uses Groovy syntax that Gradle 10 will remove. It's left as is because.release-it.jsonmatches that exact form when bumping the version.setup-gradleno longer supportsarguments:). That needs a separate fix.🤖 Generated with Claude Code