Skip to content

Omit distribution for setup-java#1198

Merged
twyatt merged 2 commits into
mainfrom
twyatt/setup-java
Jul 10, 2026
Merged

Omit distribution for setup-java#1198
twyatt merged 2 commits into
mainfrom
twyatt/setup-java

Conversation

@twyatt

@twyatt twyatt commented Jul 9, 2026

Copy link
Copy Markdown
Member

With 5.5.0 of setup-java, distribution is no longer needed:

distribution: Java distribution. Required unless java-version-file points to .sdkmanrc with a recognized distribution suffix (for example java=21.0.5-tem).


Note

Low Risk
CI-only workflow simplification with no application or security logic changes; behavior should match prior Temurin setup as long as .sdkmanrc keeps a recognized distribution suffix.

Overview
Removes the explicit distribution: temurin input from every actions/setup-java@v5 step across CI, Pages, Publish, SensorTag, and signing workflows.

Each step still uses java-version-file: .sdkmanrc, which already pins java=17.0.19-tem; setup-java 5.5.0+ infers Temurin from that suffix, so the extra distribution line is no longer required.

Reviewed by Cursor Bugbot for commit 3213921. Configure here.

@twyatt twyatt added the maintenance General maintenance that doesn't effect the public API. label Jul 9, 2026
@twyatt twyatt marked this pull request as ready for review July 9, 2026 22:51
@twyatt twyatt requested a review from a team as a code owner July 9, 2026 22:51
@twyatt twyatt requested review from cedrickcooke and sdonn3 July 9, 2026 22:51
@twyatt twyatt enabled auto-merge (squash) July 10, 2026 20:46
@twyatt twyatt merged commit 70062cf into main Jul 10, 2026
9 checks passed
@twyatt twyatt deleted the twyatt/setup-java branch July 10, 2026 21:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance General maintenance that doesn't effect the public API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants