jamesnetherton commented on PR #3959: URL: https://github.com/apache/camel-quarkus/pull/3959#issuecomment-1207845489
It looks mostly ok to me. A nitpick on the Maven Wrapper. If we are going to (at least temporarily) have the sample project use its own wrapper, we should probably make sure the version is aligned with the one used by CQ. Also, do you think we should add a dedicated docs page for this? I.e perhaps eventually it'd be part of the release process to do a perf check, so we could link to the relevant bits from the release guide. -- 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