gnodet-bot commented on code in PR #27139:
URL: https://github.com/apache/camel/pull/27139#discussion_r4146904335
##########
dsl/camel-jbang/camel-jbang-core/src/main/java/org/apache/camel/dsl/jbang/core/commands/ai/ProjectCapabilities.java:
##########
@@ -195,6 +211,16 @@ public static Capabilities build(Overview overview,
AiContent ai) {
* Plumbing, as the sources show it: only error handling (onException, a
dead letter channel) reaches the route, or
* every step of it only logs and it is not where messages enter.
*/
+ /**
+ * Whether a route starts a flow: it is an entry point (work comes in from
outside) and passes the work on to
+ * another route of the project. Such a route is business intake, never
plumbing, whatever a model says.
+ */
Review Comment:
💡 **Dangling Javadoc:** The `isUtility` Javadoc (`Plumbing, as the sources
show it...`) on the lines above is now orphaned — `startsAFlow()` was inserted
between it and `isUtility()`, so `isUtility()` has no Javadoc and the
`Plumbing...` block documents nothing.
```suggestion
/**
* Whether a route starts a flow: it is an entry point (work comes in
from outside) and passes the work on to
* another route of the project. Such a route is business intake, never
plumbing, whatever a model says.
*/
```
Then move the `Plumbing...` Javadoc block to right above `isUtility()` (line
224).
--
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]
