rahulshishodia commented on code in PR #15382:
URL: https://github.com/apache/grails-core/pull/15382#discussion_r2877992699
##########
grails-gradle/plugins/src/main/groovy/org/grails/gradle/plugin/core/GrailsExtension.groovy:
##########
@@ -106,6 +106,15 @@ class GrailsExtension {
this.indy.set(enabled)
}
+ /**
+ * Preserve method parameter names in .class files
+ * This would make grails to not have to specify '@qualifier' for wiring
by name.
+ * Additionally enables better IDE code completion for method parameters,
reflection-based tools, bean introspection, etc.).
+ * Corresponds to groovyc `-parameters`.
+ * Default: true (See #13028)
+ */
+ boolean preserveParameterNames = true
Review Comment:
@jdaugherty - To add to my previous comment. Without setter `false` would
need to be set directly using `Property#set()` like below:
```groovy
grails {
preserveParameterNames.set(false)
}
```
It works but inconsistent with other configuration. Also, since
`preserveParameterNames` is of type `Property` now, directly assignment to
`boolean` would cause `ClassCastException`.
--
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]
