codeconsole commented on PR #15259: URL: https://github.com/apache/grails-core/pull/15259#issuecomment-3607971699
> We discussed this PR in the weekly meeting. Given that the multiple properties allow for the option to customize select libraries, we want to keep that feature. > > I believe the only other feature this PR contributed was sorting the properties. If you want to update the PR for that we can merge this. Otherwise, we'll plan on closing this one. @jdaugherty can you give an actual use case where you would use these properties? There is no grouping on any of them which makes no difference if you import them directly. ```xml <grails.async.gpars.version>7.1.0-SNAPSHOT</grails.async.gpars.version> <grails.events.gpars.version>7.1.0-SNAPSHOT</grails.events.gpars.version> ``` why would you set ```properties grails.async.gpars.version=7.1.0-SNAPSHOT ``` instead of just putting it in the `buld.gradle` directly? what is the point? ``` implementation 'org.apache.grails.asyncs:grails-async-gpars:7.1.0-SNAPSHOT' ``` I could, possibly, understand `grails.async.version`, but is there really a use case for introducing 130 individual properties with the same exact version? What is the point of that? The decision to keep 130 properties sounds like it is going to introduce a lot of unknown behavior by the end user. -- 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]
