borinquenkid commented on PR #15816: URL: https://github.com/apache/grails-core/pull/15816#issuecomment-4931448727
@jdaugherty Yes — I own the full chain through the end. This is PR1 of 5 (#15816 → #15951 → #15817 → #15832 → #15833), and I intend to see all of them through to merge, not just this baseline step. Also took your point on ordering: `settings.gradle` folding is no longer deferred to the end — I split it out into its own PR2 (#15951, stacked directly on this one) so the module becomes a real subproject of root right after the baseline lands, matching how `grails-data-graphql` did it. GormRegistry wiring is now PR3, examples/docs is PR4, checkstyle cleanup is PR5. Each PR's description has an updated reading-order banner linking the whole chain. Given that, `grails7-initial-neo4jUpdates` should be safe to abandon — it predates the Groovy 5 / GormRegistry work (its merge-base is the original removal commit) and doesn't carry anything this chain doesn't already redo against the current baseline. Thanks for flagging the skill too — will take a look before the next integration like this. -- 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]
