Tarun4201 commented on PR #13406: URL: https://github.com/apache/maven/pull/13406#issuecomment-6108726376
All points from the latest re-review have been addressed in the latest push: 1. **InputLocation Level Fixed**: Scoped property locations are now nested inside the `"properties"` `InputLocation` rather than set at the top-level `ModelBase` map. This guarantees locations persist across inheritance passes in `DefaultInheritanceAssembler`. 2. **Pre-Existing Property Clashes Handled**: Added a check and SLF4J `DEBUG`-level log when an injected `profile.<id>.<key>` overrides a user-declared model property, and documented that `profile.<id>.*` properties are reserved. Added `testProfileScopePropertiesOverridesExplicitModelProperty` to lock down this precedence. 3. **Clean Rebase**: The branch is rebased cleanly as a single commit directly on top of `master`. -- 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]
