Warn about mixing logical and ternary operators

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 is priority of this? rule should warn of using brackets explicitly
obj.someValue || conditionSatisfied ? 'good' : 'bad'

// 1 option - rule should not warn
(obj.someValue || conditionSatisfied) ? 'good' : 'bad'

// 2 option - rule should not warn
obj.someValue || (conditionSatisfied ? 'good' : 'bad')

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 https://github.com/eslint/eslint/issues/7903

[karthikkp]

@mysticatea @platinumazure The changes have been made for this and a PR is open for review.

[platinumazure]

Thanks @mysticatea. In that case, I’m 👍 for this enhancement.