GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Reimagine setup-java README by brunoborges · Pull Request #1192 · actions/setup-java

Conversation

@brunoborges

@brunoborges brunoborges commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

This refresh makes the README easier to scan while keeping the technical details accurate for setup-java users. It puts common examples first, turns long input and distribution lists into tables, and clarifies where users should look for deeper guidance.

Key updates:

  • Reorganized the README around quick start, inputs, distributions, version syntax, caching, toolchains, and publishing.
  • Clarified GitHub-hosted runner tool-cache behavior, especially the Temurin pre-cache and how non-Temurin distributions usually resolve from remote metadata on fresh hosted runners.
  • Separated primary dependency cache documentation from Maven and Gradle wrapper distribution caches, including the distinct key formats and cache paths.
  • Tightened Maven vs Gradle wording so generated configuration is scoped to Maven settings/toolchains, while Gradle publishing remains build-script driven.
  • Sorted supported distributions alphabetically while keeping jdkfile last.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@brunoborges
Implement new feature for user authentication and improve error handling
e8e2a33
@brunoborges
brunoborges requested a review from a team as a code owner August 3, 2026 16:58
Copilot AI review requested due to automatic review settings August 3, 2026 16:58
Copilot started reviewing on behalf of brunoborges August 3, 2026 16:58
View session
Copilot AI reviewed Aug 3, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR refreshes the actions/setup-java README to be more scannable while preserving technical accuracy, by moving common examples earlier, converting long lists to tables, and regrouping sections around core workflows (installing, version selection, caching, toolchains, and publishing).

Changes:

  • Reorganized the README into a “quick start first” layout with a Contents section and clearer section boundaries.
  • Replaced long input/distribution lists with tables and consolidated examples for common use cases.
  • Expanded/clarified caching documentation, including wrapper caches and cache-key composition.
Show a summary per file
File Description
README.md Major documentation restructure; adds quick start examples, tables for inputs/distributions/version syntax, and revised caching/toolchain/publishing guidance.

Review details

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Suppressed comments (1)

README.md:319

  • Same as the primary cache key section: ${{ arch }} / ${{ wrapperCacheName }} look like workflow expressions but the action computes these segments internally (process.arch and wrapper cache name values like maven-wrapper). Clarifying the placeholder format would prevent users from trying to reference nonexistent expressions.
Maven and Gradle wrapper distributions are restored and saved as additional cache entries, separate from the primary dependency cache. These entries have their own keys in the form `setup-java-${{ runner.os }}-${{ arch }}-${{ wrapperCacheName }}-${{ fileHash }}`.
  • Files reviewed: 1/1 changed files
  • Comments generated: 3
  • Review effort level: Lite

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
Comment thread README.md Resolved

Uh oh!

There was an error while loading. Please reload this page.
Comment thread README.md Resolved

Uh oh!

There was an error while loading. Please reload this page.
Comment thread README.md Resolved

Uh oh!

There was an error while loading. Please reload this page.
brunoborges added 4 commits August 3, 2026 13:04
@brunoborges
Clarify README review feedback
7e154f2 
Clarify distribution case sensitivity, GHES token defaults, and cache key placeholder notation in the README.

Co-authored-by: Copilot App <[email protected]>

Copilot-Session: f0ff1923-c6ca-472e-83b2-3a813ec6d781
@brunoborges
Update README to clarify V6 development status and enhance contributi…
951678d 
…ons section
@brunoborges
Update README to recommend permissions for setup-java action in GitHu…
5a6d277 
…b Actions
@brunoborges
Restore README usage heading
9bed64b 
Rename the quick-start section to the conventional Usage heading used by setup actions.

Co-authored-by: Copilot App <[email protected]>

Copilot-Session: f0ff1923-c6ca-472e-83b2-3a813ec6d781
@brunoborges
brunoborges requested a review from Copilot August 3, 2026 17:28
Copilot started reviewing on behalf of brunoborges August 3, 2026 17:28
View session
Copilot AI reviewed Aug 3, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review details

  • Files reviewed: 1/1 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@brunoborges
brunoborges merged commit 7c6f629 into actions:main Aug 3, 2026
3 checks passed

Uh oh!

There was an error while loading. Please reload this page.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@brunoborges
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.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.