jeanouii commented on code in PR #1541:
URL: https://github.com/apache/activemq/pull/1541#discussion_r2608512117


##########
activemq-mqtt/pom.xml:
##########
@@ -293,6 +294,70 @@
         </plugins>
       </build>
     </profile>
+    <profile>
+     <id>parallel-tests</id>
+     <activation>
+       <property>
+         <name>parallel-tests</name>
+       </property>
+     </activation>
+     <build>
+       <plugins>
+         <plugin>
+           <artifactId>maven-surefire-plugin</artifactId>

Review Comment:
   I'd agree and had it in the first place to be honest. 
   I always put the groupId for consistency and avoid having sometimes implicit 
versus explicit values.
   Maven 4 or 4.1 now prints a warning when there is a duplicate plugin 
defined. I'm not sure it would work if you have one using implicit and another 
one using explicit.



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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
For further information, visit: https://activemq.apache.org/contact


Reply via email to