FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

docs: update setup-java examples by HarithaVattikuti · Pull Request #1131 · actions/setup-java · GitHub

docs: update setup-java examples - #1131

Merged
brunoborges merged 4 commits into
actions:mainfrom
HarithaVattikuti:latestversions
Jul 16, 2026
Merged

docs: update setup-java examples#1131
brunoborges merged 4 commits into
actions:mainfrom
HarithaVattikuti:latestversions

Conversation

HarithaVattikuti commented Jul 15, 2026
edited
Loading

Copy link
Copy Markdown
Contributor

Updated the documentation examples in README.md and docs/advanced-usage.md to use actions/checkout@v7 and actions/setup-java@v6. Also removed the obsolete V2 vs V1 section from the README.

Validation: docs-only change.

Copilot AI review requested due to automatic review settings July 15, 2026 22:10
HarithaVattikuti requested a review from a team as a code owner July 15, 2026 22:10

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Pull request overview

This PR updates documentation examples for actions/setup-java by bumping referenced action versions and removing an obsolete README section, aiming to keep user-facing workflow snippets current.

Changes:

  • Update workflow examples to use actions/checkout@v7 and actions/setup-java@v6
  • Remove the obsolete “V2 vs V1” section from README.md

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
README.md Updates workflow snippets to newer action versions and removes the legacy migration section.
docs/advanced-usage.md Updates many advanced workflow examples to newer action versions.

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

Comment thread docs/advanced-usage.md Outdated

Copy link
Copy Markdown
Contributor

Hey @HarithaVattikuti good finding! Just wondering if we should time this after the v6 release? Should we push that release by end of the month? I'd like to push v5.6 before that.

Copy link
Copy Markdown
Contributor Author

Hey @HarithaVattikuti good finding! Just wondering if we should time this after the v6 release? Should we push that release by end of the month? I'd like to push v5.6 before that.

Thanks @brunoborges — that makes sense. Since this PR is specifically for v6 docs, I think we should merge it alongside the v6 release. Let’s push v5.6 first, then merge this closer to the v6 release

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 190c65c4-f5aa-4595-8e2e-26266fbcb37f
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 190c65c4-f5aa-4595-8e2e-26266fbcb37f
brunoborges merged commit 46f6294 into actions:main Jul 16, 2026
3 checks passed

Copy link
Copy Markdown

👋 Hi @HarithaVattikuti and @brunoborges, this update broke the README.md rendering for Usage and Maven options, and adds unnecessary spacing in all touched files. Can we revert this change and only update the necessary parts?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL