no-extra-semi does not handle empty if blocks

[fb55]

What version of ESLint are you using?

2.12.0

What parser (default, Babel-ESLint, etc.) are you using?

Default

Please show your full configuration:

{"extends": "eslint:recommended"}

What did you do? Please include the actual source code causing the issue.

if(arr[0] === "1");
else if(arr[0] === "2"){
    ...
}

What did you expect to happen?

Nothing.

What actually happened? Please include the actual, raw output from ESLint.

  31:48  error  Unnecessary semicolon  no-extra-semi

Also, when using --fix, the semicolon is removed, which leads to invalid code.

[michaelficarra]

This rule simply warns on all empty statements. You can use an empty block ({}) or some effect-free expression (0) instead.

[fb55]

That's right, but --fix still produces invalid code, which IMHO should never happen.

[michaelficarra]

You're right, that is an issue.

[mysticatea]

No, that rule has exceptions by the parent node type:

eslint/lib/rules/no-extra-semi.js

Line 69 in cd1b057

allowedParentTypes = ["ForStatement", "ForInStatement", "ForOfStatement", "WhileStatement", "DoWhileStatement"];

But the exception list does not include IfStatement.
Maybe it should include IfStatement.