Skip to content

Conversation

@jord1e
Copy link
Contributor

@jord1e jord1e commented Feb 1, 2022

Gradle 7.3 is required for building Java 17 applications

See: https://docs.gradle.org/7.3/release-notes.html#java17

Someone needs to take a look at the BND plugin because in the current configuration it won't work for Gradle 8.x: https://github.com/bndtools/bnd/wiki/Changes-in-6.0.0#bnd-gradle-plugins

Part of my CI effort (#2676 )

@jord1e
Copy link
Contributor Author

jord1e commented Feb 3, 2022

Looks like the checks passed, first one timed out whilst validating the wrapper, weird

@bbakerman
Copy link
Member

Looks like the checks passed, first one timed out whilst validating the wrapper, weird
We are seeing builds time out more and more on GitHub of late

@jord1e
Copy link
Contributor Author

jord1e commented Feb 3, 2022

Seems to be a Cloudflare/Gradle issue
gradle/wrapper-validation-action#40

I will extract the wrapper validation logic into a separate action like OpenTelemetry:
https://github.com/open-telemetry/opentelemetry-java-instrumentation/blob/main/.github/workflows/gradle-wrapper-validation.yml

@andimarek andimarek merged commit 9c77ddd into graphql-java:master Feb 3, 2022
@andimarek andimarek added this to the 18.0 milestone Feb 3, 2022
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