-
Notifications
You must be signed in to change notification settings - Fork 89
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Plugin should not use the deprecated GUtil.loadProperties() function #378
Comments
This was referenced Sep 7, 2022
Merged
bot-gradle
added a commit
to gradle/gradle
that referenced
this issue
Sep 10, 2022
…from the org.gradle.util package * #21839 For some members of `org.gradle.util.*` deprecation logging could not be added because smoke tested plugins and other third parties don't offer a version that is free of their use: * **Kotlin Gradle Plugin** * `WrapUtil.toDomainObjectSet()` * [KT-53882 - Kotlin Gradle Plugin should not use the deprecated WrapUtil.toDomainObjectSet() function](https://youtrack.jetbrains.com/issue/KT-53882/Kotlin-Gradle-Plugin-should-not-use-the-deprecated-WrapUtiltoDomainObjectSet-function) * #21882 * **Android Gradle Plugin** * `GUtil.toWords(string)` * [ANDROID-245405989 - Android Gradle Plugin should not use the deprecated GUtil.toWords(string) function](https://issuetracker.google.com/issues/245405989) * #21899 * `ConfigureUtil.configure(closure, target)` * [ANDROID-245405989 - Android Gradle Plugin should not use the deprecated ConfigureUtil.configure(closure, target) function](https://issuetracker.google.com/issues/245405994) * #21901 * **Spring Dependency Management Plugin** * `GradleVersion.getNextMajor()` * spring-gradle-plugins/dependency-management-plugin#338 * #21898 * **Google Protobuf Gradle Plugin** * `GUtil.toCamelCase(string)` * google/protobuf-gradle-plugin#604 * #21900 * `ConfigureUtil.configure(closure, target)` * google/protobuf-gradle-plugin#577 * #21901 * **OSDetector Gradle Plugin (via the Protobuf plugin)** * `VersionNumber.*` * google/osdetector-gradle-plugin#24 * #21902 * **Asciidoctor Gradle Plugin / Grolifant** * `CollectionUtils.stringize(collection)` * [GROLIFANT-85 - Grolifant should not use the deprecated CollectionUtils](https://gitlab.com/ysb33rOrg/grolifant/-/issues/85) * #21903 * **Nebula Lint Plugin** * `GUtil.loadProperties(inputStream)` * nebula-plugins/gradle-lint-plugin#378 * #21912 * **Nebula Dependency Lock Plugin** * `NameMatcher` * nebula-plugins/gradle-dependency-lock-plugin#243 * #21913 * **IntelliJ IDEA** * `GUtil.toCamelCase(string)` * `GUtil.toLowerCamelCase(string)` * `GUtil.loadProperties(inputStream)` * `ConfigureUtil.configureByMap(properties, delegate)` * [IDEA-301430 - IntelliJ IDEA should not use deprecated types from the org.gradle.util package](https://youtrack.jetbrains.com/issue/IDEA-301430/IntelliJ-IDEA-should-not-use-deprecated-types-from-the-orggradleutil-package) * #21912 * #21900 * #21914 Co-authored-by: Bo Zhang <[email protected]>
FYI we plan to start nagging in Gradle 8.1 |
Hi, sorry for the super late response. This is not used anymore 👍 since 18.0.x |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
In Gradle 7.1,
org.gradle.util.GUtil
has been soft deprecated (@Deprecated
annotation).For Gradle 7.6 we wanted to add deprecation nagging to it. Our smoke tests show that the Nebula Lint Plugin still uses the
GUtil.loadProperties()
function. We will postpone adding the deprecation nagging because users currently cannot resolve it.Could you please look into removing that usage from the Nebula Lint Plugin?
The text was updated successfully, but these errors were encountered: