📎 Implement lint/noEmptyBlockStatements - eslint/no-empty eslint/no-empty-static-block

[Conaclos]

Description

This lint rule should integrate both no-empty and no-empty-static-block rules.

Want to contribute? Lets you know you are interested! We will assign you to the issue to prevent several people to work on the same issue. Don't worry, we can unassign you later if you are no longer interested in the issue! Read our contributing guide and analyzer contributing guide.

[ematipico]

What's the difference between this rule and #42? It's worth an explanation in the description

[Conaclos]

What's the difference between this rule and #42? It's worth an explanation in the description

In contrast to #42, the rule prohibits any empty block statements.
Examples of incorrect code:

if (cond) {}

while (cond) {}

function f() {}

However, the following example also triggers #42:

{}

[ematipico]

So we would have two rules that trigger the same code? That doesn't seem right to me.

I wonder if at this point if we should have only one rule that handles all the block statements. Or at least make sure that they don't conflict each other.

[Conaclos]

I wonder if at this point if we should have only one rule that handles all the block statements

I think you are right. We could thus regroup these two rule under the noUselessBlockStatements name.

[andrewangelle]

Hello! If a fix for this issue is still needed, I'd like to give it a shot.