You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@camel.apache.org by GitBox <gi...@apache.org> on 2022/03/31 08:10:16 UTC

[GitHub] [camel-quarkus] aldettinger commented on pull request #3686: Camel-quarkus-datasonnet extension - resolves Issue #2078

aldettinger commented on pull request #3686:
URL: https://github.com/apache/camel-quarkus/pull/3686#issuecomment-1084242307


   @javaduke Great to see that an extension is almost there :)
   
   I wonder if the build issue on CI would be linked to inverting the order of bom imports. That switch might have consequences, we'll need to find an answer here.
   
   Apart from that, a curiosity, the PR embeds dslibs.jar in the system under test for "datasonnet functionaly testing" purpose. Is it really needed to embed this ? For instance, couldn't it just be a test scoped dependency ?


-- 
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-unsubscribe@camel.apache.org

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