davsclaus commented on code in PR #26586: URL: https://github.com/apache/camel/pull/26586#discussion_r4046209084
########## components/camel-once/src/main/docs/once-component.adoc: ########## @@ -40,7 +40,7 @@ include::partial$component-endpoint-headers.adoc[] == Example -To set up a route that generates an event every 60 seconds: +To set up a route that generates a single event at the start of the route: Review Comment: The same sentence also lives in the generated catalog copy of this page, `catalog/camel-catalog/src/generated/resources/org/apache/camel/catalog/docs/once-component.adoc` (line 43), which still says `every 60 seconds`. That file is regenerated from this one during the build, and CI fails on uncommitted generated changes if it is not included. Could you run a build of `catalog/camel-catalog` (e.g. `mvn -DskipTests install` in that module after installing `components/camel-once`) and commit the regenerated file too? Optional wording nit: since the component description is "Trigger a single message only once at startup", something like `generates a single event once when Camel starts` might read a bit more precisely than "at the start of the route" — but either is fine. -- 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]
