Prevent confusing if-statement usage

[willsoto]

As per the suggestion here, I am opening up a an issue to discuss extending the rules around if blocks.

The blocks were originally indented like so:

if (condition) {
  // stuff
} if (anotherUnrelatedCondition) {
  // more
} 

I found this confusing because at first glance it looks like if; else if blocks.

---

I would image the rule could be something like:

ensure 1 newline between if blocks

Other suggestions are welcome.

[dcousens]

That really should be disallowed. The wriggle room for accidents is pretty high.

[KhaledMohamedP]

Totally agree here. It should be disallowed 👍

[feross]

Agreed. This should be prevented. However, there's no eslint rule that allows us to catch this case.

I just opened an issue to get the eslint team's thoughts: eslint/eslint#7116

[feross]

Closing this, as we're blocked. I'll re-open if the ESLint issue makes any progress.