Gradle upgrade to 8.2 #52
Conversation
kranthise
commented
Aug 11, 2023
kranthise
commented
- Also some third party library upgrades
|
|
||
| test.setRuntimeClasspath(project.files( | ||
| test.getOutput(), | ||
| main.getOutput(), | ||
| project.getConfigurations().getByName(JavaPlugin.TEST_RUNTIME_CONFIGURATION_NAME), | ||
| gosuRuntime.inferGosuClasspath(project.getConfigurations().getByName(JavaPlugin.TEST_COMPILE_CONFIGURATION_NAME)))); | ||
| project.getConfigurations().getByName(JavaPlugin.TEST_RUNTIME_ONLY_CONFIGURATION_NAME), |
There was a problem hiding this comment.
Consider using testRuntimeClasspath that automatically combines testRuntime and testImplementation,
see https://docs.gradle.org/current/userguide/java_plugin.html#sec:java_plugin_and_dependency_management (scroll down to see the diagrams of configuration dependencies)
Also there might be an error when you try to resolve TEST_RUNTIME_CONFIGURATION_NAME as it is not resolvable by default, while testRuntimeClasspath is resolvable. Example of this behavior https://stackoverflow.com/questions/47910578/not-able-to-copy-configurations-dependencies-after-upgrading-gradle-plugin-for-a
There was a problem hiding this comment.
Updated the configuration. Thank you
src/main/java/org/gosulang/gradle/tasks/compile/GosuCompile.java
Outdated
Show resolved
Hide resolved
kranthise
force-pushed
the
user/kg/gradle8-upgrade
branch
from
52656b1
to
50ada88
Compare
|
This seems out of date; closing as we are using Gradle 8.6 already. |
