borinquenkid commented on PR #15816: URL: https://github.com/apache/grails-core/pull/15816#issuecomment-4886362158
Thanks for flagging this, James — you're right that this PR doesn't touch `settings.gradle`. Neo4j and graphql were both moved into this repo standalone (`0e04cf1463`) as a holding pattern while they hadn't caught up to the current baseline. graphql has since finished that migration and was folded into root `settings.gradle` for real; this PR is the neo4j equivalent of that first step — get it compiling and passing tests against core-impl's GORM (`8.0.0-SNAPSHOT`) while still a separate build, then wire it into GormRegistry (PR2, already up), then fold it into root `settings.gradle` the same way graphql was (PR3). I've written up that PR3 scope in [`grails-data-neo4j/GORM_REGISTRY_MIGRATION.md`](https://github.com/apache/grails-core/blob/feat/neo4j-gorm-registry-wiring/grails-data-neo4j/GORM_REGISTRY_MIGRATION.md) on this branch so it's tracked rather than implied. -- 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]
