dhruv9b commented on PR #1024:
URL: https://github.com/apache/maven-enforcer/pull/1024#issuecomment-5648083486

   
   > Perfect! Thanks!
   
   Thanks again for the question — I looked further into Maven's report plugin 
resolution to make sure the implementation matches Maven's behavior.
   
   I found that the effective reporting plugin version is resolved in this 
order: an explicitly configured `<reporting>` version, then `<build><plugins>`, 
then `<build><pluginManagement>`.
   
   My initial implementation only covered the explicit reporting version and 
`pluginManagement`. I’ve now updated the PR to also check `build/plugins` 
before `pluginManagement`, and added a unit test to verify that precedence.
   
   I also verified the original #916 reproducer with the Maven Invoker 
integration test, which now passes successfully.
   


-- 
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]

Reply via email to