Update: add option "allowInParentheses" to no-sequences (fixes #14197) #14199
What is the purpose of this pull request? (put an "X" next to an item)
[ ] Documentation update
What rule do you want to change?
Does this change cause the rule to produce more or fewer warnings?
How will the change be implemented? (New option, new default behavior, etc.)?
new option "allowInParentheses".
Please provide some example code that this change will affect:
What changes did you make? (Give an overview)
Add new option "allowInParentheses" to rule "no-sequences" to opt-in for allowing parenthesized sequence expressions.
Is there anything you'd like reviewers to focus on?
This is the first commit which adds the option and adapts the unit tests.
To be done:
The text was updated successfully, but these errors were encountered:
What about a second option "allowForLoopAfterthoughts" (name taken from no-plusplus)? Can/should be
Would make the last sentence in the documentation obsolete ("Another case is where you might want to report all usages of the comma operator, even in a for loop. You can achieve this using rule
The CLA check is still pending. It looks like you used a different email address for commits, so maybe this could help: