Skip to content

Fix GradleWrapperTest after Gradle 9.4.0 got released#6871

Merged
timtebeek merged 1 commit intomainfrom
greg-gradle-wrapper-test
Mar 4, 2026
Merged

Fix GradleWrapperTest after Gradle 9.4.0 got released#6871
timtebeek merged 1 commit intomainfrom
greg-gradle-wrapper-test

Conversation

@greg-at-moderne
Copy link
Contributor

@greg-at-moderne greg-at-moderne commented Mar 4, 2026

What's changed?

Adding 9.4.0 to known Gradle versions as it has been released earlier today.

What's your motivation?

Fix broken CI, where GradleWrapperTest fails with:

GradleWrapperTest > listGradleWrappers() FAILED
    java.lang.NullPointerException: Cannot invoke "org.openrewrite.gradle.internal.GradleWrapperScriptLoader$Version.getGradlewChecksum()" because "this.resolved" is null
        at org.openrewrite.gradle.internal.GradleWrapperScriptLoader$Nearest.gradlew(GradleWrapperScriptLoader.java:69)

@greg-at-moderne greg-at-moderne self-assigned this Mar 4, 2026
@github-project-automation github-project-automation bot moved this to In Progress in OpenRewrite Mar 4, 2026
@greg-at-moderne greg-at-moderne marked this pull request as ready for review March 4, 2026 12:40
@github-project-automation github-project-automation bot moved this from In Progress to Ready to Review in OpenRewrite Mar 4, 2026
@timtebeek timtebeek merged commit fda3bb6 into main Mar 4, 2026
1 check passed
@timtebeek timtebeek deleted the greg-gradle-wrapper-test branch March 4, 2026 12:51
@github-project-automation github-project-automation bot moved this from Ready to Review to Done in OpenRewrite Mar 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants