Adopt Gradle 8 #322
Adopt Gradle 8 #322
Conversation
* Update dependency org.spekframework.spek2:spek-runner-junit5 to v2.0.19 * Update dependency org.spekframework.spek2:spek-dsl-jvm to v2.0.19 * Group Spek2 updates --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Goooler <wangzongler@gmail.com>
* Update dependency gradle to v8 * Use bin * Migrate pluginBundle * Remove org.gradle.daemon flag * Remove forUseAtConfigurationTime * Use current Gradle version in tests * Update plugin org.jetbrains.kotlin.jvm to v1.9.10 * Fix WordSpecTest * Generate wrapper --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Goooler <wangzongler@gmail.com>
* Update circleci/openjdk Docker tag to v11 * Use toolchain * cimg/openjdk:17.0 * Apply org.gradle.toolchains.foojay-resolver-convention --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Goooler <wangzongler@gmail.com>
|
We can rebase these commits and replace @dependabot with @renovate-bot. |
|
@Goooler thanks for this pull request. Can you justify why are you switching to renovate bot? I much prefer to use dependabot as it is built into GitHub now. |
@renovate-bot can send the Gradle updates, and supports more kinds of dep updates. |
|
Thanks. Lastly, is this PR going to be compatible with Gradle 7? |
It should work, maybe we should add more Gradle versions into test matrix. |
|
Did a quick test at Goooler#27, seems need to fix some tests. Edited: Not sure why these tests failed, but should not be relevant to the removal of |
|
No problem, looks like some classes introduced in 7.6 are being used in your branch indirectly. I'll do some more checks later and merge this. Thanks once again. |
* Use compileOnly * Remove extra localGroovy() * Defer tasks * Use pluginManagement & dependencyResolutionManagement * Migrate deprecated buildDir * Always publish scans on CI * Remove mustRunAfter * Add jvmargs * Add .gitattributes
|
Thanks once again for a great PR |
Closes #321.