The GitHub Actions job "Code Style" on 
grails-core.git/fix/16567-datasource-properties-yaml has failed.
Run started by GitHub user jamesfredley (triggered by jamesfredley).

Head commit for run:
84e8282825533ba5b386c338ac17173c7bad45b7 / James Fredley 
<[email protected]>
Keep source precedence when rebuilding dataSource.properties

The map is now rebuilt from each property source in precedence order,
so the highest source wins for every path whatever its shape:

- A scalar or map from a higher source is not replaced by a lower
  nested key, map, or unresolved placeholder.
- A map stored at the exact path is read per source instead of being
  returned early, so higher flattened keys still win over it.
- Indexed keys such as connectionInitSqls[0] build a List, copying an
  existing list instead of mutating it.
- Literal dotted keys such as oracle.jdbc.flag are kept as written and
  block the same nested path from lower sources and from later entries
  of the same source.
- A string the conversion service turns into a map is used at its own
  precedence, after placeholder resolution.

String values still go through Environment.getProperty, so the
unresolvable-placeholder setting is kept.

ConfigurationBuilderSpec now covers the Environment path, including a
map-backed @Builder(SimpleStrategy) type and an unchanged
DatastoreUtils resolver. The datasource specs cover a YAML list and the
dotted application.groovy form.

Assisted-by: Sisyphus:grok-4.7
Assisted-by: Sisyphus:claude-opus-5-5

Report URL: https://github.com/apache/grails-core/actions/runs/40835645915

With regards,
GitHub Actions via GitBox

Reply via email to