amondel2 commented on code in PR #432:
URL:
https://github.com/apache/grails-intellij-plugin/pull/432#discussion_r4191386347
##########
IMPROVEMENT-PLAN.md:
##########
@@ -195,6 +200,16 @@ Detection must work from Gradle dependency data (the
plugin already has a
init,utils,assets}` plus `src/main/groovy`, `src/test/groovy`,
`src/integration-test/groovy` (registered by `TestPhasesGradlePlugin`)
and
`src/functional-test/groovy` (functional/Geb phase) source sets.
+ **Ground truth (grails-core `8.0.x`, checked 2026-10-01):** the
generated web skeleton is
Review Comment:
Fixed in `e1ad4a0`. It had `grails-app/{…,utils,…}`, so `utils` generated
and `init/` not — the inverse of the truth. grails-forge's `GrailsApplication`
feature writes `grails-app/init/{packagePath}/Application.groovy` and
`BootStrap.groovy`, so `init/` **is** generated, and `GrailsBase` creates
**no** `grails-app/utils`; that picture comes from the legacy
`grails-profiles/web/skeleton`, whose `base` variant still contributes
`grails-app/init/.../Application.groovy`. Rewritten.
The `migrations` half, including the
`grails.plugin.databasemigration.changelogLocation` gap, is correct and I left
it byte-identical.
--
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]
