arturobernalg opened a new pull request, #1075: URL: https://github.com/apache/maven-enforcer/pull/1075
Fixes #1074. RequireSameVersions.execute merged the per-source (dependency, build plugin, report plugin) results with Map.putAll, so a version shared by two sources had the earlier source's members silently replaced by the later one's in the failure message. Detection was unaffected, only the message was incomplete. collectVersionMembers now writes directly into the shared map instead of returning a separate map to be merged with putAll. Testing: added a regression test that fails on the old code (the dependency member disappears from the message) and passes with the fix. Ran the full enforcer-rules test suite plus checkstyle/spotless, all green. --- To make clear that you license your contribution under the [Apache License Version 2.0, January 2004](https://www.apache.org/licenses/LICENSE-2.0) check one of the following: - [x] I hereby declare this contribution to be licensed under the [Apache License Version 2.0, January 2004](https://www.apache.org/licenses/LICENSE-2.0) - [x] I hereby declare that I have filed an [Apache Individual Contributor License Agreement](https://www.apache.org/licenses/icla.pdf). -- 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]
