jdaugherty commented on PR #15407: URL: https://github.com/apache/grails-core/pull/15407#issuecomment-3931144431
I'd suggest we move this to draft status then. I'm a -1 because of the burden of keeping this up to date when defaults are added/changed, documenting external plugins that aren't even in core (changes external to core will mismatch historical documentation), & the generated doc assumes the same features for every project (if different libraries are included, these defaults can change, be missing, or even wrong). The chance of being wrong here is what I view as the largest detriment. A compromise proposal: 1. Change this PR so that the current default values are extracted from core & only include libraries that will exist in every Grails app. i.e. no hibernate, no database migration, no asset pipeline, nothing optional. Store those defaults in a way they can be retrieved programmatically. 2. Use the branch I created to determine what's overridden at build time in a test app 3. Take the output from 1 & 2 to generate a config doc that we publish with a disclaimer that the configuration values are the base settings that may be changed based on additional project settings. 4. Update the documentation to point people to the command so they know how to get the specific settings for their project Longer term we can move those defaults into a location that we can programmatically retrieve them (I gave one proposal earlier). -- 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]
