Skip to content

[ci] Fix version number of gradle-update/update-gradle-wrapper-action#3273

Merged
juherr merged 1 commit into
testng-team:masterfrom
gruenich:feature/ci-fix-gradle-actions
May 1, 2026
Merged

[ci] Fix version number of gradle-update/update-gradle-wrapper-action#3273
juherr merged 1 commit into
testng-team:masterfrom
gruenich:feature/ci-fix-gradle-actions

Conversation

@gruenich

@gruenich gruenich commented Apr 30, 2026

Copy link
Copy Markdown
Contributor

Also adjust gradle/actions/wrapper-validation to v6

Summary by CodeRabbit

  • Chores
    • Updated Gradle wrapper validation configurations across build and publishing workflows to use newer tooling versions, enhancing the robustness of the continuous integration pipeline.

@coderabbitai

coderabbitai Bot commented Apr 30, 2026

Copy link
Copy Markdown
📝 Walkthrough

Walkthrough

GitHub Actions workflows were tweaked to update action versions: two publish workflows switch the Gradle wrapper validation action from v3 to v6, and the update-gradle-wrapper workflow changes the gradle-update action reference from v6 to v2. No other workflow logic was changed.

Changes

Cohort / File(s) Summary
Publish workflows
.github/workflows/publish-maven-central.yml, .github/workflows/publish-snapshot.yml
Bumped gradle/actions/wrapper-validation action from @v3 to @v6. No other step inputs or logic changed.
Wrapper update workflow
.github/workflows/update-gradle-wrapper.yml
Changed action reference from gradle-update/update-gradle-wrapper-action@v6 to gradle-update/update-gradle-wrapper-action@v2.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • juherr
  • krmahadevan

Poem

🐰
Gradle leaps and trims its fur,
Actions shift, the workflows purr,
Versions tuned with nimble hop,
CI hums and never stops. ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: updating the gradle-update/update-gradle-wrapper-action version number from v6 to v2 due to a name scheme change in the CI workflows.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
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 unit tests (beta)
  • Create PR with unit tests

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.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

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/update-gradle-wrapper.yml:
- Line 15: The workflow currently uses the validation action (uses:
gradle/actions/wrapper-validation@v6) which only validates the wrapper; replace
that step in the "Update Gradle Wrapper" job with the official update action
(gradle-update/gradle-update-action@vX) so it creates PRs to bump the wrapper.
Update the step "uses" reference from gradle/actions/wrapper-validation to
gradle-update/gradle-update-action, add required inputs (e.g., GITHUB_TOKEN or
repository-token) per the action docs, and keep any existing schedule/branch
settings so the job opens an automatic update PR instead of only validating.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 261751d5-c80c-461e-abd0-ce28c99082bd

📥 Commits

Reviewing files that changed from the base of the PR and between ccc766a and 488e361.

📒 Files selected for processing (3)
  • .github/workflows/publish-maven-central.yml
  • .github/workflows/publish-snapshot.yml
  • .github/workflows/update-gradle-wrapper.yml

Comment thread .github/workflows/update-gradle-wrapper.yml Outdated

@juherr juherr left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Fix the wrapper update action

Also adjust gradle/actions/wrapper-validation to v6
@gruenich gruenich force-pushed the feature/ci-fix-gradle-actions branch from 488e361 to e936471 Compare April 30, 2026 14:40
@gruenich gruenich changed the title [ci] Properly replace gradle-update/update-gradle-wrapper-action [ci] Fix version number of gradle-update/update-gradle-wrapper-action Apr 30, 2026
@gruenich gruenich requested a review from juherr April 30, 2026 15:26

- name: Update Gradle Wrapper
uses: gradle-update/update-gradle-wrapper-action@v6
uses: gradle-update/update-gradle-wrapper-action@v2

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I don't get why the action if downgraded from v6 to v2

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

No v6 exists. That why the execution of the action failed tonight: https://github.com/testng-team/testng/actions/runs/25146310886
I wanted to fix it this morning and wrongly replaced the action. Now it should be working again (failing with the issue as before).

@juherr juherr merged commit e371374 into testng-team:master May 1, 2026
9 of 10 checks passed
@gruenich gruenich deleted the feature/ci-fix-gradle-actions branch May 1, 2026 07:56
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.

3 participants