java7: modernise workflow (replace archived wrapper-validation action)#422
Merged
kplattret merged 5 commits intoJun 22, 2026
Merged
Conversation
gradle/wrapper-validation-action is archived and superseded by gradle/actions/wrapper-validation. Switch both usages in java7.yml to the new action, pinned to v6.2.0. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
The 'adopt' (AdoptOpenJDK) distribution alias is deprecated in actions/setup-java; Eclipse Temurin is its successor (and what java.yml already uses). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Java 7 compatibility is enforced at compile time via targetCompatibility = '1.7', and Java 8 is the lowest JDK that can run Gradle and execute the tests, so a real JDK 7 test run isn't possible. Add a comment so the matrix isn't "fixed" later by adding Java 7. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Modernises the java7 GitHub Actions workflow by updating deprecated/archived CI actions and clarifying the rationale for the Java runtime used in CI, without changing the java7/ library itself.
Changes:
- Replace archived
gradle/wrapper-validation-actionwithgradle/actions/wrapper-validationpinned tov6.2.0in both jobs. - Switch
actions/setup-javadistribution from deprecatedadopttotemurin. - Add an explanatory comment documenting why the CI matrix runs on Java 8 only.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
dili91
approved these changes
Jun 22, 2026
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.
Modernises
.github/workflows/java7.yml. Workflow-only — no changes to thejava7/library.Changes
gradle/wrapper-validation-actionwith its official successorgradle/actions/wrapper-validation, pinned to v6.2.0 (both the build-and-test and publish jobs). The old action's repo is archived and points here.temurindistribution instead of the deprecatedadoptalias inactions/setup-java(matchesjava.yml).targetCompatibility = '1.7', and Java 8 is the lowest JDK that can both run Gradle and execute the tests (Gradle requires Java 8+ to run, and its toolchains don't support Java 7 as a target) — so a real JDK 7 test run isn't possible. The comment prevents a future well-intentioned "add Java 7" change that would hit the Gradle wall.Note
This intentionally does not bump java7's Gradle (8.0.2) or add multi-runtime testing — that would be a larger, coordinated change to the
java7/build files, best as its own PR.🤖 Generated with Claude Code