Make no-case-declarations rule fixable

When error is spotted, fix will be about wrapping case clauses with blocks. It’ll be great to have it.

---

What rule do you want to change?

no-case-declarations

Does this change cause the rule to produce more or fewer warnings?

No

How will the change be implemented? (New option, new default behavior, etc.)?

Fix functionality will be configured

Please provide some example code that this change will affect:

Code as:

switch (foo) {
case "bar":
  let bar;
}

Will be fixed to:

switch (foo) {
case "bar": {
  let bar;
}
}

What does the rule currently do for this code?

There’s no fix functionality configured

What will the rule do after it’s changed?

Fix functionality will be configured

[medikoo]

@ljharb Sure indentation should be maintained, this fix should be purely about adding block wrap nothing more (I’ve updated example)

[not-an-aardvark]

Thanks for your interest in improving ESLint. Unfortunately, it looks like this issue didn’t get enough support from the team and so I’m closing it. While we wish we’d be able to accommodate everyone’s requests, we do need to prioritize. We’ve found that issues failing to reach consensus after a long time tend to never do it, and as such, we close those issues. This doesn’t mean the idea isn’t interesting, just that it’s not something the team can commit to.