yesamer commented on code in PR #6748:
URL: 
https://github.com/apache/incubator-kie-drools/pull/6748#discussion_r3371451435


##########
kie-parent/pom.xml:
##########
@@ -1596,6 +1604,77 @@
         <artifactId>jetty-jakarta-servlet-api</artifactId>
         <version>${version.org.eclipse.jetty.jakarta.servlet.api}</version>
       </dependency>
+      <!-- Explicit Jetty 12.0.33 dependency management to prevent version 
conflicts -->

Review Comment:
   @ChinchuAjith consider you're importing the bom, it's not necessary to 
declare the below dependencies here, those should be already imported by the 
above bom. Please check here if all dependencies are declared in the bom:  
https://repo1.maven.org/maven2/org/eclipse/jetty/jetty-bom/12.1.10/jetty-bom-12.1.10.pom,
 but I assume the bom already contains all of them, so this block is not 
necessary



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

Reply via email to