ppalaga commented on a change in pull request #226: Use MainSupport as base for running Camel URL: https://github.com/apache/camel-quarkus/pull/226#discussion_r331400183
########## File path: extensions/core/deployment/src/main/java/org/apache/camel/quarkus/core/deployment/CamelSupport.java ########## @@ -18,13 +18,26 @@ import java.io.IOError; import java.io.IOException; +import java.io.InputStream; import java.lang.reflect.Modifier; import java.nio.file.Files; import java.nio.file.Path; +import java.util.ArrayList; +import java.util.Collection; +import java.util.HashSet; +import java.util.List; +import java.util.Map; +import java.util.Properties; +import java.util.Set; import java.util.stream.Stream; import io.quarkus.deployment.builditem.ApplicationArchivesBuildItem; +import org.apache.camel.RoutesBuilder; +import org.apache.camel.builder.AdviceWithRouteBuilder; +import org.apache.camel.builder.RouteBuilder; import org.jboss.jandex.ClassInfo; +import org.jboss.jandex.DotName; +import org.jboss.jandex.IndexView; public final class CamelSupport { Review comment: A minimal JavaDoc stating the purpose of this class would be nice. If the public methods are supposed to be usable by other extensions, they should get a minimal JavaDoc too. Those ones not supposed to be publicly usable should not be public (but a single sentence describing the contract and behavior would still be nice, although it is much less important for private and package-visible methods). ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services