jamesnetherton commented on code in PR #8946:
URL: https://github.com/apache/camel-quarkus/pull/8946#discussion_r3813088573
##########
extensions-support/langchain4j/deployment/pom.xml:
##########
@@ -52,6 +52,11 @@
<artifactId>quarkus-junit-internal</artifactId>
<scope>test</scope>
</dependency>
+ <dependency>
+ <groupId>io.quarkiverse.langchain4j</groupId>
+ <artifactId>quarkus-langchain4j-core-deployment</artifactId>
+ <optional>true</optional>
+ </dependency>
Review Comment:
`<optional>true</optional>` can't work here.
If you run a test where QL4J is not present like `langchain4j-agent`, then
you get:
```
ClassNotFoundException:
io.quarkiverse.langchain4j.deployment.ExcludeFromImpliedAiServiceBuildItem
```
The only way around it is to make the dependency non-optional. But then
you'd need to also make the runtime dependency also non-optional. That'd mean
QL4J is always on the runtime classpath whether you want it or not.
--
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]
