Skip to content

Update Gradle Usage in CI#34

Merged
thebeanogamer merged 2 commits intomasterfrom
gradlefix
Jan 26, 2026
Merged

Update Gradle Usage in CI#34
thebeanogamer merged 2 commits intomasterfrom
gradlefix

Conversation

@thebeanogamer
Copy link
Member

No description provided.

Copilot AI review requested due to automatic review settings January 26, 2026 16:43
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates GitHub Actions workflows to use the newer Gradle setup action and run Gradle tasks via the wrapper during CI/release publishing.

Changes:

  • Replace gradle/gradle-build-action usage in the release workflow with gradle/actions/setup-gradle@v5 plus direct ./gradlew task execution.
  • Update the main Gradle CI workflow to use gradle/actions/setup-gradle@v5 instead of a pinned commit SHA.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
.github/workflows/release.yaml Switches release publishing steps to setup-gradle@v5 and direct ./gradlew commands.
.github/workflows/gradle.yml Updates the build job’s Gradle setup action reference to @v5.

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

Copy link
Contributor

@amilbourne amilbourne left a comment

Choose a reason for hiding this comment

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

CoPilot seems to be a bit out of date on what the actions do as the setup action should now do validation.

@thebeanogamer thebeanogamer merged commit 5b706e4 into master Jan 26, 2026
10 checks passed
@thebeanogamer thebeanogamer deleted the gradlefix branch January 26, 2026 22:32
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