[GitHub] [maven-dependency-plugin] slawekjaranowski commented on a change in pull request #195: [MDEP-788] Upgrade maven-reporting-impl to version 3.1.0

[GitBox <gi...@apache.org>]

slawekjaranowski commented on a change in pull request #195:
URL: https://github.com/apache/maven-dependency-plugin/pull/195#discussion_r805599521



##########
File path: src/it/projects/analyze-report/pom.xml
##########
@@ -59,13 +59,40 @@
       <plugins>
         <plugin>
           <artifactId>maven-dependency-plugin</artifactId>
-          <configuration>
-           <outputXML>true</outputXML>
-           <scriptableOutput>true</scriptableOutput>
-           <verbose>true</verbose>
-          </configuration>
+          <version>@project.version@</version>
+        </plugin>
+        <plugin>
+          <artifactId>maven-site-plugin</artifactId>
+          <version>3.10.0</version>

Review comment:
       This is IT test ... project also override m-site-plugin from parent
    
   In such situation will be good to have some possibility to inherit plugin version from parent pom, in other case IT test will have outdated dependency in some time and will be not maintain
   
   So I prefer to update m-site-p in separate PR




-- 
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: issues-unsubscribe@maven.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org