Eslint rules

Enforce semis and remove unneeded padded blocks

The padded blocks rule makes it so

if (a) {

    b();

}

becomes

if (a) {
    b();
}

We need something for nested ternary functions too. As in ternarys in ternarys.

merged

mentioned in commit 0065e6f2

I added "no-nested-ternary": "warn", in a previous merge, which should give a warning when there is a ternary in a ternary. All good, or should I remove it?

Remove, ternary in ternary is fine, it just needs to be well formatted or else they will become confusing messes.

Oh, unless you are talking about a single ternary nested in a check. Those are 100% fine and needed in some cases.

I do not like them being used for strings in printed text though. That annoys me.

You can always suppress warnings for a single line if nested ternaries are absolutely needed, so there's that.

A rule to detect =+ may be a good idea.