vlsi edited a comment on pull request #2563: URL: https://github.com/apache/calcite/pull/2563#issuecomment-934072635
Thank you for looking into this, however, I would like to refrain from making the change now. Currently, the task definition and configuration are co-located, so they are easier to discover and understand. If you move a half of the configuration to a different file, then it becomes harder to reason about. I'm going to merge Gradle 7.2 migration soon, and then we'll be able to use https://docs.gradle.org/current/userguide/structuring_software_products.html approach, so virtually all the code from the root file would be split across feature plugins. -- 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]
