Copilot commented on code in PR #14757:
URL: https://github.com/apache/grails-core/pull/14757#discussion_r2096395370
##########
grails-gradle/plugins/build.gradle:
##########
@@ -35,7 +35,7 @@ dependencies {
// TODO: tasks are isolated as of Gradle 7 so we must expose the version
of the groovy.
// We could upgrade to groovy 4 for the tasks, but gradle plugins are not
isolated and still
// need groovy 3.
- implementation "org.codehaus.groovy:groovy"
+ compileOnly 'org.codehaus.groovy:groovy'
Review Comment:
Switching the dependency to 'compileOnly' may require downstream modules to
explicitly include a runtime Groovy dependency. Consider documenting this
change or ensuring that consumers of this plugin are aware of the need to
provide Groovy at runtime.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
