davsclaus commented on PR #28586: URL: https://github.com/apache/camel/pull/28586#issuecomment-6467547759
Thanks for the review, @oscerd. - **Conflict:** resolved. The branch merges the latest #28585 (17a98fa1a6fd), so it is mergeable again. - **CI:** the "Build and test" workflow only runs for PRs that target `main`, so a stacked PR gets only `dependency-review` and `upload`. The full build and `license-check` run once #28575 and #28585 are merged and this PR is retargeted to `main`. Until then, run locally at the top of the stack: camel-diagram 197 tests (with `RouteDiagramCalloutTest`), and the route-diagram, history and error command tests of camel-jbang-core with `RouteDiagramProblemsTest` (59), all pass. - **Since your review:** a `RuntimeException` from `SourceValidator` or `RouteAssist` now also becomes a problem of the file on its first step (cc61e3a0771f), like a file that cannot be read, so a checker bug on a broken file does not fail the diagram. _Claude Code on behalf of davsclaus_ -- 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]
