jdaugherty commented on code in PR #14080: URL: https://github.com/apache/grails-core/pull/14080#discussion_r2008076202
########## RENAME.md: ########## @@ -0,0 +1,150 @@ +# Grails Repackaging Reference + +As part of the move to the Apache Software Foundation, the grails artifacts must adhere to the apache namespace. Below +is a reference of all migrated artifacts - both their old and new name. + +| old group id | old artifact id | new group id | new artifact id | old gradle plugin name | new gradle plugin name | repository | +|-----------------------------------|-------------------------------------------------|-------------------------------------------------|---------------------------------------------------------------|------------------------|------------------------|------------------------| +| org.grails.plugins | async | org.apache.grails | grails-async | | | grails-core | +| org.grails | grails-async | org.apache.grails.async | grails-async-core | | | grails-core | +| org.grails | grails-async-gpars | org.apache.grails.async | grails-async-gpars | | | grails-core | +| org.grails | grails-async-rxjava | org.apache.grails.async | grails-async-rxjava | | | grails-core | +| org.grails | grails-async-rxjava2 | org.apache.grails.async | grails-async-rxjava2 | | | grails-core | +| org.grails | grails-async-rxjava3 | org.apache.grails.async | grails-async-rxjava3 | | | grails-core | +| org.grails | grails-bom | org.apache.grails | grails-bom | | | grails-core | +| org.grails | grails-bootstrap | org.apache.grails.bootstrap | grails-bootstrap | | | grails-core | +| org.grails.plugins | cache | org.apache.grails | grails-cache | | | grails-cache | +| org.grails | grails-plugin-codecs | org.apache.grails | grails-codecs | | | grails-core | +| org.grails | grails-codecs | org.apache.grails.codecs | grails-codecs-core | | | grails-core | +| org.grails | grails-console | org.apache.grails | grails-gradle-console | | | grails-core | +| org.grails | grails-plugin-controllers | org.apache.grails | grails-controllers | | | grails-core | +| org.grails.plugins | converters | org.apache.grails | grails-converters | | | grails-core | +| org.grails | grails-core | org.apache.grails | grails-core | | | grails-core | +| org.grails.plugins | hibernate5 | org.apache.grails | grails-data-hibernate5 | | | grails-data-mapping | +| org.grails.plugins | database-migration | org.apache.grails | grails-data-hibernate5-dbmigration | | | grails-data-mapping | +| org.grails | gorm-hibernate5-spring-boot | org.apache.grails | grails-data-hibernate5-spring-boot | | | grails-data-mapping | +| org.grails.plugins | hibernate6 | org.apache.grails | grails-data-hibernate6 | | | grails-data-hibernate6 | +| org.grails.plugins | database-migration | org.apache.grails | grails-data-hibernate6-dbmigration | | | grails-data-hibernate6 | +| org.grails | gorm-hibernate6-spring-boot | org.apache.grails | grails-data-hibernate6-spring-boot | | | grails-data-hibernate6 | +| org.grails | grails-datastore-gorm-async | org.apache.grails.data | grails-data-mapping-async | | | grails-data-mapping | +| org.grails | grails-datastore-gorm | org.apache.grails.data | grails-data-mapping-core | | | grails-data-mapping | +| org.grails | grails-datastore-gorm-hibernate5 | org.apache.grails.data | grails-data-mapping-hibernate5 | | | grails-data-mapping | +| org.grails | grails-datastore-gorm-hibernate6 | org.apache.grails.data | grails-data-mapping-hibernate6 | | | grails-data-hibernate6 | +| org.grails | grails-datastore-gorm-mongodb | org.apache.grails.data | grails-data-mapping-mongodb | | | grails-data-mapping | +| org.grails | grails-datastore-gorm-mongodb-ext | org.apache.grails.data | grails-data-mapping-mongodb-ext | | | grails-data-mapping | +| org.grails | grails-datastore-gorm-support | org.apache.grails.data | grails-data-mapping-support | | | grails-data-mapping | +| org.grails | tck-base | org.apache.grails.data | grails-data-mapping-tck-base | | | grails-data-mapping | +| org.grails | tck-domains | org.apache.grails.data | grails-data-mapping-tck-domains | | | grails-data-mapping | +| org.grails | tck-tests | org.apache.grails.data | grails-data-mapping-tck-tests | | | grails-data-mapping | +| org.grails | grails-gorm-testing-support | org.apache.grails.testing | grails-data-mapping-testing-support | | | grails-data-mapping | Review Comment: We did not realize this until after we refactored all of the build files, but one of the largest problems was including test dependencies on the production classpath of many grails apps. One of the reasons I pushed to have a `.testing` namespace was to ensure that doesn't happen going forward. In your example, we have several of these testing support libraries: grails-views-gson-testing-support grails-testing-support grails-web-testing-support grails-datamapping-testing-support Shouldn't we follow the pattern on all of these then? Would this work for you? org.apache.grails.testing:grails-testing-support-views-gson org.apache.grails.testing:grails-testing-support org.apache.grails.testing:grails-testing-support-web org.apache.grails.testing:grails-testing-support-datamapping -- 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: commits-unsubscr...@grails.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org