davsclaus commented on code in PR #29931:
URL: https://github.com/apache/camel/pull/29931#discussion_r4573490333
##########
dsl/camel-jbang/camel-jbang-core/src/main/java/org/apache/camel/dsl/jbang/core/commands/Debug.java:
##########
@@ -622,13 +625,37 @@ protected int doRunDebug(KameletMain main) throws
Exception {
return 0;
}
- private void removeDebugOnlyOptions(List<String> cmds) {
- // only check Debug.class (not super classes)
- RunHelper.doWithFields(Debug.class, fc -> cmds.removeIf(c -> {
- String n1 = "--" + fc.getName();
- String n2 = "--" + StringHelper.camelCaseToDash(fc.getName());
- return c.startsWith(n1) || c.startsWith(n2);
- }));
+ /**
+ * The arguments without the options of camel debug, which camel run does
not have: each by its exact name, with its
+ * value (--breakpoint=tick, or --breakpoint tick). Before, every argument
that started with the name of a field was
+ * removed, such as --source-dir=... for the field source, and the value
of --breakpoint tick was left behind as a
+ * file (CAMEL-26930).
Review Comment:
Done in 12392fd6fb5f, in shorter sentences: what the method does first, then
what went wrong before (CAMEL-26930) in a paragraph of its own.
_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]
