New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Warn about mixing logical and ternary operators #11704
Warn about mixing logical and ternary operators #11704
Comments
Thank you for your proposal. I like this idea. |
Hi @mieszko4, thanks for the issue. Is this not currently possible with |
The current |
Thanks @mysticatea. In that case, I'm |
@mysticatea @platinumazure The changes have been made for this and a PR is open for review. |
What rule do you want to change?
no-mixed-operators
Does this change cause the rule to produce more or fewer warnings?
more
How will the change be implemented? (New option, new default behavior, etc.)?
new option with default behaviour to warn
Please provide some example code that this change will affect:
What does the rule currently do for this code?
it does not warn user about mixed logical and ternary expressions
What will the rule do after it's changed?
warn to use brackets around to make it explicit what the priority is
Are you willing to submit a pull request to implement this change?
nope, with this issue, I am just checking if this would make sense
Might be related to #7903
The text was updated successfully, but these errors were encountered: