ppalaga commented on code in PR #4592:
URL: https://github.com/apache/camel-quarkus/pull/4592#discussion_r1113882843
##########
extensions/zendesk/runtime/pom.xml:
##########
@@ -51,6 +51,12 @@
<dependency>
<groupId>org.apache.camel</groupId>
<artifactId>camel-zendesk</artifactId>
+ <exclusions>
+ <exclusion>
+ <artifactId>async-http-client</artifactId>
+ <groupId>org.asynchttpclient</groupId>
+ </exclusion>
+ </exclusions>
Review Comment:
Why is this exclusion needed? If it there to solve dependency convergence
issues, then I'd rather vote for managing `async-http-client` in our BOM. In
that way, we can make sure that all our extensions (and tests) using it will
use the same version which is better for our and end users' reproducibility.
--
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: commits-unsubscr...@camel.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
