-
Notifications
You must be signed in to change notification settings - Fork 82
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
Upgrade to Gradle 8.2 #840
Conversation
@DRSchlaubi do you have an idea how we can get rid of this warning?
|
What is that even related to? useJunitPlatform()? |
it's already applied in the convention plugin |
The changes in gradle-wrapper.jar, gradle-wrapper.properties and gradlew are the result of running this command twice: ./gradlew wrapper --gradle-version 8.2 --gradle-distribution-sha256-sum 38f66cd6eef217b4c35855bb11ea4e9fbc53594ccccb5fb82dfd317ef8c2c5a3 This commit also includes: * removing no longer needed systemProp.org.gradle.unsafe.kotlin.assignment=true, see https://docs.gradle.org/8.2/release-notes.html#simple-property-assignment-in-kotlin-dsl-enabled-by-default * add org.gradle.kotlin.dsl.allWarningsAsErrors=true, see https://docs.gradle.org/8.2/release-notes.html#fail-on-script-compilation-warnings
hm, it's just a warning, what about just ignoring it for now? (since i don't really understand it and don't know how to fix it) |
maybe gradle 8.2.1 fixes it? |
included these changes in #842 |
This isn't a Gradle issue, KGP just doesn't use the jvm-test-suite plugin, so kt has to fix it |
ok 👍 |
The changes in
gradle-wrapper.jar
,gradle-wrapper.properties
andgradlew
are the result of running this command twice:This PR also includes:
systemProp.org.gradle.unsafe.kotlin.assignment=true
, see https://docs.gradle.org/8.2/release-notes.html#simple-property-assignment-in-kotlin-dsl-enabled-by-defaultorg.gradle.kotlin.dsl.allWarningsAsErrors=true
, see https://docs.gradle.org/8.2/release-notes.html#fail-on-script-compilation-warnings