[eslint] change operator-linebreak option for ternary operators

[golopot]

Current configuration enforces ternary expressions to be written like:

a ?
b :
c

This is the opposite of the popular style. It is probably a mistake introduced during the time when this project was being converted to Airbnb style.

[ljharb]

if this line is removed, isn’t this what the Airbnb config already mandates?

[golopot]

Airbnb config uses "before".

https://github.com/airbnb/javascript/blob/master/packages/eslint-config-airbnb-base/rules/style.js#L422

[ljharb]

oh right. how much would change if we switched to that? Alternately, could we use before here, and use explicit overrides for the ones we want set to after, to match the Airbnb config?

[golopot]

There will be 840 lines of codes changed if that said option were to be switched to Airbnb style.
golopot@2678544
I would like to see the option being kept as is, due to my personal preference for "after" style, and for the sake of avoiding git diffs in history.

[ljharb]

I prefer the "before" style personally; but even if we wanted to continue setting operators like && to "after", that seems better to me (ie, to be overriding airbnb's "before" setting for specific operators, instead of replacing them)

[golopot]

I suppose the next action is you come up with an configuration and I will accept it. I believe you.

[ljharb]

I'm going ahead with the "before" style by removing the override.