SQL operators can be one or more characters #3842
Comments
|
If it is an operator, w probably don't need more than three of these. Also, as we are using it in more than one place, probably it is better to extract them and use it in an include file, so at least we use the same everywhere. Good find. |
|
I meant this rule and 942120, for example. |
|
@fzipi should I replace the corresponding PR for this with |
|
Yes sounds more reasonable. But then the real refactor should come from the inclusion, right? Meaning: it doesn't matter too much because if we change it for all the operators in their right form, there is no need for "counting" 1..3 😄 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is there a reason for not using
[|&<>*\/%=^+-]+in the following regex?coreruleset/regex-assembly/942520.ra
Line 15 in a6036b9
The text was updated successfully, but these errors were encountered: