davsclaus commented on PR #26824:
URL: https://github.com/apache/camel/pull/26824#issuecomment-5810000745
The doc build was still failing after my escaping, and the escaping was the
reason — `\${body}` does not escape anything. Asciidoctor reads the attribute
reference from the brace, so the backslash has to sit in front of *that*:
`$\{body}`. The rest of the Camel documentation writes it that way 32 times; my
form appeared zero times, which was the clue I should have taken first.
Fixed in `7cd6da4878fd`, and @gnodet's latest regen is merged — note it
propagated my broken form into the catalog copy rather than correcting it, so
both files are now on `$\{` and byte-identical.
Two other things from chasing it, in case they are useful to anyone reading
later:
* `docs/components/modules/languages/pages/simple-operators.adoc`, which the
warning names, is a symlink to the source under `core/camel-core-languages`, so
the source is the only place to fix. The catalog copy is a real generated file
and does need syncing.
* The three "failing" builds before this were the `maven build` step
**cancelled**, not failed — the workflow's concurrency group cancels an
in-progress run when a new push lands, and I had pushed twice in quick
succession.
--
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]
