opensearch-project / opensearch-sdk-java

OpenSearch SDK to build and run extensions
Apache License 2.0
28 stars 59 forks source link

chore(deps): update dependency gradle to v8.2.1 #855

Closed mend-for-github-com[bot] closed 1 year ago

mend-for-github-com[bot] commented 1 year ago

This PR contains the following updates:

Package Update Change
gradle (source) minor 8.1.1 -> 8.2.1

Release Notes

gradle/gradle ### [`v8.2.1`](https://togithub.com/gradle/gradle/releases/tag/v8.2.1): 8.2.1 This is a patch release for 8.2. We recommend using 8.2.1 instead of 8.2 It fixes the following issues: - [#​25579](https://togithub.com/gradle/gradle/issues/25579) Regression in 8.2: StackOverflowError w/ Gradle 8.2 + Quarkus 2.16.7 (latest) - [#​25611](https://togithub.com/gradle/gradle/issues/25611) TestKit unexpectedly stopped working with Gradle 2.x versions - [#​25618](https://togithub.com/gradle/gradle/issues/25618) Micronaut JacocoReportAggregationPlugin broken in Gradle 8.2 - [#​25658](https://togithub.com/gradle/gradle/issues/25658) Gradle 8.2 sets incorrect value to boolean --no-feature option - [#​25674](https://togithub.com/gradle/gradle/issues/25674) Address regression in dependency graph build operation results [Read the Release Notes](https://docs.gradle.org/8.2.1/release-notes.html) #### Upgrade Instructions Switch your build to use Gradle 8.2.1 by updating your wrapper: ./gradlew wrapper --gradle-version=8.2.1 See the [Gradle 8.x upgrade guide](https://docs.gradle.org/8.2.1/userguide/upgrading_version\_8.html#changes\_8.2) to learn about deprecations, breaking changes and other considerations when upgrading to Gradle 8.2.1. #### Reporting Problems If you find a problem with this release, please file a bug on [GitHub Issues](https://togithub.com/gradle/gradle/issues) adhering to our issue guidelines. If you're not sure you're encountering a bug, please use the [forum](https://discuss.gradle.org/c/help-discuss).

Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


dbwiddis commented 1 year ago

Gradle is rightfully calling out a 8.2 deprecated feature (JavaPluginConventions class in opensearch gradle plugin) that will be removed in Java 9, but I don't understand why it's failing the build in any gradle 8.x.

https://docs.gradle.org/8.2/userguide/upgrading_version_8.html#deprecated_access_to_conventions

dbwiddis commented 1 year ago
org.gradle.api.GradleException: Deprecated Gradle features were used in this build, making it incompatible with Gradle 9.0
        at org.gradle.internal.featurelifecycle.LoggingDeprecatedFeatureHandler.displayDeprecationIfSameMessageNotDisplayedBefore(LoggingDeprecatedFeatureHandler.java:90)
        at org.gradle.internal.featurelifecycle.LoggingDeprecatedFeatureHandler.featureUsed(LoggingDeprecatedFeatureHandler.java:75)
        at org.gradle.internal.deprecation.DeprecationLogger.nagUserWith(DeprecationLogger.java:347)
        at org.gradle.internal.deprecation.DeprecationLogger.nagUserWith(DeprecationLogger.java:246)
        at org.gradle.internal.deprecation.DeprecationMessageBuilder$WithDocumentation.nagUser(DeprecationMessageBuilder.java:130)
        at org.gradle.api.internal.project.DefaultProject.getConvention(DefaultProject.java:593)
mend-for-github-com[bot] commented 1 year ago

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

Warning: custom changes will be lost.