ppalaga commented on PR #5117: URL: https://github.com/apache/camel-quarkus/pull/5117#issuecomment-1688162593
Thanks @claudio4j, I have done another round of review. I have send a PR against your PR https://github.com/claudio4j/camel-quarkus/pull/1 with some minor fixes. Please feel free to amend it to your commit. After those fixes, out of the 4 new integration tests, 3 are passing in native mode while `camel-quarkus-integration-test-camel-k-runtime` fails. I did not have time to investigate why exactly. We might perhaps skip it on CI for now and file a follow up issue. I am still looking whether we could merge the ITs into fewer Maven modules to save some time on the CI. Great work, @claudio4j! -- 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]
