Skip to content

Update build.yml GitHub Actions to latest versions#1518

Merged
dannyroberts merged 1 commit intomasterfrom
dmr/update-ci-action-versions
Feb 17, 2026
Merged

Update build.yml GitHub Actions to latest versions#1518
dannyroberts merged 1 commit intomasterfrom
dmr/update-ci-action-versions

Conversation

@dannyroberts
Copy link
Member

@dannyroberts dannyroberts commented Feb 17, 2026

Technical Summary

Upgrade GitHub Actions in build.yml to their latest versions:

  • actions/checkout v3 → v6
  • actions/setup-java v3 → v5
  • gradle/gradle-build-action v2 → gradle/actions/setup-gradle v5 (the old action is deprecated)

Safety Assurance

Safety story

CI-only change — no impact on application code. The updated actions are backwards compatible and the build workflow behavior is unchanged.

Automated test coverage

The build workflow itself runs gradle build which includes all tests and checkstyle. A successful CI run on this PR validates the action upgrades.

QA Plan

  • Verify CI passes on this PR

Special deploy instructions

  • This PR can be deployed after merge with no further considerations.

Rollback instructions

  • This PR can be reverted after deploy with no further considerations.

Review

  • The set of people pinged as reviewers is appropriate for the level of risk of the change.

Duplicate PR

Automatically duplicate this PR as defined in contributing.md.

Summary by CodeRabbit

  • Chores
    • Updated GitHub Actions workflow with latest action versions and improved build configuration.

Upgrade actions/checkout v3→v6, actions/setup-java v3→v5, and
migrate from deprecated gradle/gradle-build-action v2 to
gradle/actions/setup-gradle v5.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@coderabbitai
Copy link

coderabbitai bot commented Feb 17, 2026

Walkthrough

GitHub Actions workflow updated to use newer action versions. Checkout bumped from v3 to v6, setup-java from v3 to v5. Gradle build action replaced with dedicated setup-gradle step (v5), with build execution moved to direct gradle command invocation.

Changes

Cohort / File(s) Summary
CI/CD Workflow Updates
.github/workflows/build.yml
Updated action versions: checkout (v3→v6), setup-java (v3→v5). Replaced gradle-build-action (v2) with setup-gradle (v5). Gradle build now executes directly via CLI instead of through action wrapper.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Workflow hops to v6 and beyond,
Where checkout and java now bond,
Gradle's new step keeps builds so spry,
Actions updated—CI speeds by!

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: updating GitHub Actions versions in build.yml, which is the primary focus of the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into master
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch dmr/update-ci-action-versions

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In @.github/workflows/build.yml:
- Line 32: Replace the deprecated distribution value used by setup-java v5:
locate the YAML key "distribution: 'adopt'" and change it to "distribution:
'temurin'" so the workflow uses Eclipse Temurin instead of the old AdoptOpenJDK
distribution; verify the surrounding setup-java step (the action for setup-java
v5) still references the correct version and run CI to confirm the job picks up
the new distribution.

uses: actions/setup-java@v5
with:
java-version: ${{ env.JAVA_VERSION }}
distribution: 'adopt'
Copy link

@coderabbitai coderabbitai bot Feb 17, 2026

Choose a reason for hiding this comment

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

🛠️ Refactor suggestion | 🟠 Major

Migrate distribution from 'adopt' to 'temurin'.

Since you're already upgrading setup-java to v5, this is a good time to address the deprecated distribution. AdoptOpenJDK got moved to Eclipse Temurin and won't be updated anymore. It is highly recommended to migrate workflows from adopt to temurin to keep receiving software and security updates.

Proposed fix
       - name: Set up Java
         uses: actions/setup-java@v5
         with:
           java-version: ${{ env.JAVA_VERSION }}
-          distribution: 'adopt'
+          distribution: 'temurin'
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
distribution: 'adopt'
distribution: 'temurin'
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In @.github/workflows/build.yml at line 32, Replace the deprecated distribution
value used by setup-java v5: locate the YAML key "distribution: 'adopt'" and
change it to "distribution: 'temurin'" so the workflow uses Eclipse Temurin
instead of the old AdoptOpenJDK distribution; verify the surrounding setup-java
step (the action for setup-java v5) still references the correct version and run
CI to confirm the job picks up the new distribution.

Copy link
Member Author

Choose a reason for hiding this comment

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

@shubham1g5 This suggestion about distribution: adopt => distribution: temurin sounds legitimate. I'm not acting on it because I don't have the context to evaluate it, but I wanted to flag it to you in case you want to act on it.

Copy link
Contributor

Choose a reason for hiding this comment

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

thanks, think we will be fine updating this as long as it builds successfully i.e I don't think there is a specific reason to use adopt here.

Copy link

Choose a reason for hiding this comment

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

Seems like the humans are having a chat. I'll hop back into my burrow for now. If you need me again, just tag @coderabbitai in a new comment, and I'll come hopping out!

@dannyroberts dannyroberts marked this pull request as ready for review February 17, 2026 14:51
@dannyroberts dannyroberts merged commit 503b601 into master Feb 17, 2026
2 of 3 checks passed
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.

2 participants

Comments