template-curly-spacing fix should not remove comments inside the curly braces.

[yeonjuan]

Tell us about your environment

• ESLint Version: v6.8.0
• Node Version: v12.13.0
• npm Version: v6.12.0

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

default

Please show your full configuration:

Configuration

module.exports = {
 "parserOptions": { "ecmaVersion": 6 },
  rules: {
   "template-curly-spacing": ["error", "never"]
  }
}

What did you do? Please include the actual source code causing the issue, as well as the command that you used to run ESLint.

• .eslintrc.js

module.exports = {
 "parserOptions": { "ecmaVersion": 6 },
  rules: {
   "template-curly-spacing": ["error", "never"]
  }
}

• test.js

const foo = `  ${  bar  /*comment*/}   `;

Run eslint with --fix option.

eslint --fix test.js

What did you expect to happen?

It should not remove comments(/*comment*/) even it has an error.

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

const foo = `  ${bar}   `;

It removes comments inside the curly braces.{,}. Here is a demo

Are you willing to submit a pull request to fix this bug?

Yes, If it is a bug.

[mdjermanovic]

It's very possible that some other spacing rules have the same bug.

Spacing rules that don't have this bug treat comments as tokens.

That's one option, another is to defer to a new space-around-comment rule (as @mysticatea mentioned in this comment) in these situations.