oscerd commented on code in PR #26733: URL: https://github.com/apache/camel/pull/26733#discussion_r4083098844
########## docs/user-manual/modules/ROOT/pages/camel-4x-upgrade-guide-4_23.adoc: ########## @@ -2580,3 +2580,14 @@ Routes that reference the constants (for example `setHeader(MustacheConstants.MU are unaffected. Routes that set the header by its literal string name, or that use `allowTemplateFromHeader=true` with the old header names, must switch to the new `Camel`-prefixed names. +=== camel-debezium - a failed embedded engine is now reported Review Comment: Correct, and my fault — the conflict resolution joined the two sides with a single newline and ate the blank line. Fixed in 0eb2056. I also swept the whole file: three other `===` headings have the same problem — lines 741 (`camel-azure-eventgrid`), 1156 (`camel-langchain4j`) and 1200 (`camel-salesforce`) — and all three are pre-existing on `main`, so I left them alone rather than widen this PR. Worth a follow-up if the convention is being enforced. Separately, the CI failure on the previous head was not this PR: `camel-jbang-core` failed to compile on `CatalogDocExamplesTest:100` (`cannot find symbol: variable m`). That was base breakage from CAMEL-24917, already fixed on `main` by fa8ab2a7a4b2, and the branch was behind it. This push is rebased onto current `main`, so it carries the fix. _Claude Code on behalf of oscerd_ -- 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]
