ClassDeclaration/ClassExpression option for the indent rule

Tell us about your environment

• ESLint Version: v3.17.1 (using the indent-rewrite branch)
• Node Version: v7.7.2
• npm Version: v4.1.2

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

Please show your full configuration:

module.exports = {
    root: true,
    parserOptions: {
        ecmaVersion: 8
    },
    rules: {
        indent: [2, 2, {
            SwitchCase: 2,
            VariableDeclarator: 2,
            outerIIFEBody: 2,
            MemberExpression: 2,
            FunctionDeclaration: {
                parameters: 2,
                body: 2
            },
            FunctionExpression: {
                parameters: 2,
                body: 2
            },
            CallExpression: {
                arguments: 2
            },
            ArrayExpression: 2,
            ObjectExpression: 2
        }]
    }
};

What did you do? Please include the actual source code causing the issue.

class a {
    m() {
        "hmm";
    }
}
let b = class {
    m() {
        "hmm";
    }
};

What did you expect to happen? There should be an option to set the indentation for ClassDeclaration and ClassExpression.

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

c:\nodejs\test\test.js
  2:5  error  Expected indentation of 2 spaces but found 4  indent
  3:9  error  Expected indentation of 6 spaces but found 8  indent
  4:5  error  Expected indentation of 2 spaces but found 4  indent
  7:5  error  Expected indentation of 2 spaces but found 4  indent
  8:9  error  Expected indentation of 6 spaces but found 8  indent
  9:5  error  Expected indentation of 2 spaces but found 4  indent

✖ 6 problems (6 errors, 0 warnings)

$ eslint test.js --fix Lints to:

class a {
  m() {
      "hmm";
  }
}
let b = class {
  m() {
      "hmm";
  }
};

[not-an-aardvark]

It would be better to depend on lodash, ast-utils, and functional-red-black-tree directly in your plugin rather than loading ESLint’s versions of the modules at runtime. That way, your plugin won’t break if ESLint modifies its dependencies or helper functions in a future release.

[nemoxps]

@mqklin That error is caused by this line: const lodash = require('../../eslint/node_modules/lodash'); To explain this line: I’m assuming that ESLint and my plugin both are installed globally. It is in my case and it just works fine. If you need it to work locally, just fill an issue and I’m going to fix it.