gnodet-bot commented on code in PR #28714:
URL: https://github.com/apache/camel/pull/28714#discussion_r4421296613
##########
core/camel-core-languages/src/main/java/org/apache/camel/language/simple/SimpleSyntaxHints.java:
##########
@@ -82,7 +82,8 @@ public final class SimpleSyntaxHints {
private static final String[] OPERATOR_WORDS = {
"==", "!=", ">=", "<=", ">", "<", "=~", "!=~", "~~", "!~~",
"contains",
- "!contains", "regex", "!regex", "in", "!in", "is", "!is", "range",
"!range", "startsWith", "endsWith", "&&", "||" };
+ "!contains", "regex", "!regex", "in", "!in", "is", "!is", "range",
"!range", "startsWith", "!startsWith",
+ "endsWith", "!endsWith", "equals", "!equals", "&&", "||" };
Review Comment:
💡 The public `OPERATORS` constant (lines 29–32, not in this diff) lists
valid operators in human-readable error messages (lines 138, 143, 163). After
this PR it's missing the four new operators: `!startsWith`, `!endsWith`,
`equals`, `!equals`. Please update `OPERATORS` to keep it in sync with
`OPERATOR_WORDS`.
--
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]
