Rule Change: Add suggestion fixer to no-case-declarations

[JoshuaKGoldberg]

What rule do you want to change?

no-case-declarations

What change do you want to make?

Implement suggestions

How do you think the change should be implemented?

A new default behavior

Example code

/*eslint no-case-declarations: "error"*/

switch (foo) {
  case 1:
    let x = 1;
    break;
}

What does the rule currently do for this code?

No fixers or suggestions, per #15851.

What will the rule do after it's changed?

Add a suggestion to wrap the case 1's consequent in {}:

/*eslint no-case-declarations: "error"*/

switch (foo) {
  case 1: {
    let x = 1;
    break;
  }
}

Participation

• I am willing to submit a pull request to implement this change.

Additional comments

The discussion in

[mdjermanovic]

Add a suggestion to wrap the case 1's consequent in {}:

Makes sense to me 👍

This is exactly what the doccumentation for this rule is suggesting - examples of incorrect code are fixed by adding {} in examples of correct code.

@eslint/eslint-team thoughts?

Additional comments

The discussion in

Looks like something is missing here? :)

[aladdin-add]

👍

[nzakas]

Makes sense. Go for it @JoshuaKGoldberg. 👍

[JoshuaKGoldberg]

Looks like something is missing here? :)

Blurgh, just noticed this. It's a very annoying GitHub bug (https://twitter.com/JoshuaKGoldberg/status/1765397127675498870).

There was some discussion somewhere and I ... don't remember where. Blugh.