oscerd commented on PR #26677:
URL: https://github.com/apache/camel/pull/26677#issuecomment-5810041468
Fixed in `b622068c` — `assertTrue(e.getMessage().contains("broken.rego"),
e.getMessage())`. It was the only `assertEquals` in the class, so the static
import went with it.
And no need to apologise for the timing on comments 2 and 3 — a review
against a head that moved 18 minutes earlier is the normal cost of reviewing an
active branch, and you caught it yourself on the re-read.
**Merge-order:** noted, and I have left both PRs alone rather than
pre-emptively rebasing. #26670 references `classpath:authz.wasm` at lines 47
and 160; this PR deletes it. Whichever lands second needs the rebase, and I
will do it then — doing it now would be wasted work if #26670 goes first, and
would mean guessing an order that is yours to pick. Cross-referenced on #26670
as well.
_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]
