Erroneous indentation warning in template literals contents

Tell us about your environment

• ESLint Version: 5.6.1

• Node Version: 8.12.0

• npm Version: 6.4.1

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

Please show your full configuration:

Configuration

    module.exports = {
        "env": {
            "browser": true,
            "es6": true
        },
        "extends": "eslint:recommended",
        "parserOptions": {
            "ecmaVersion": 2018,
            "sourceType": "module"
        },
        "rules": {
            "indent": [
                "error",
                2
            ],
            "linebreak-style": [
                "error",
                "unix"
            ],
            "quotes": [
                "error",
                "single"
            ],
            "semi": [
                "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.

class ShopCart {
  render () {
    const { html, map } = this
    return html`
      <style>
        :host { display: block; }
      </style>
      ${map((item) =>
        item++
      )}
    `
  }
}

new ShopCart 

./eslint.js -c config.js test.js

What did you expect to happen?

No errors

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

I expected it to go through without any warnings. However, I got this output:

/home/merc/.atom/packages/linter-eslint/node_modules/eslint/bin/test.js
   9:1  error  Expected indentation of 4 spaces but found 8  indent
  10:1  error  Expected indentation of 2 spaces but found 6  indent

✖ 2 problems (2 errors, 0 warnings)
  2 errors and 0 warnings potentially fixable with the `--fix` option.

Please note that the right indentation for that item++ is 8, not 4. Same applies to the closing of the map() function.

[mercmobily]

OK – will try that. I have to ask though… what are the chances of this getting looked at? Please realise that it’s giving the wrong warnings for template literals; I humbly ask you to consider to turn this into a bug, rather than an enhancement. It makes it painful to use libraries that use template literals as template engines (HyperHTML, Lit-html, etc.)

[eslint-deprecated[bot]]

Unfortunately, it looks like there wasn’t enough interest from the team or community to implement this change. 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 accepted status after 21 days tend to never be accepted, and as such, we close those issues. This doesn’t mean the idea isn’t interesting or useful, just that it’s not something the team can commit to.

Thanks for contributing to ESLint and we appreciate your understanding.