Rule option request: multiline-ternary to disallow newlines

[ljharb]

The multiline-ternary rule allows me to require ternaries to be multiline - I'd like an option that requires ternaries not be multiline.

What version of ESLint are you using?
v3.1.1

What rule do you want to change?
multiline-ternary, I'd like a "disallowMutliline" option added.

What code should be flagged as incorrect with this change?

var x = a ?
  b :
  c;

var y = a
  ? b
  : c;

var z = a ? b :
  c;

What happens when the rule is applied to this code now?
It passes.

[vitorbal]

Thanks, @ljharb, I know it can be a bit tedious, but would you mind filling in the template for proposing a rule change?

[ljharb]

@vitorbal ah, sorry, i didn't realize there was a template for that as well. Updated.

[mysticatea]

I set JSCS Compatibility milestone since JSCS has disallowMultiLineTernary rule.

[kaicataldo]

Interestingly enough, this was discussed and we ended up deciding to only implement "always". I'm happy to add this in if/when it's accepted. Assigning myself now so I don't lose track of it.

[kaicataldo]

@eslint/eslint-team Thoughts? Looks like we need two more thumbs on this.

[kaicataldo]

For context: #6066 (comment)

[ilyavolodin]

I'll support this. Seems like a reasonable enhancement to multiline-ternary. 👍

[platinumazure]

👍 from me.

[ilyavolodin]

@kaicataldo If you are championing this, I think you can mark it as accepted at this point.