indent rule: Invalid indentation for array in sequencing function calls

What version of ESLint are you using?

2.13.1

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

default

Please show your full configuration:

• Operation system Linux arch 4.5.4-1-ARCH #1 SMP PREEMPT Wed May 11 22:21:28 CEST 2016 x86_64 GNU/Linux
• Node.js version v4.4.3

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

Source code:

function f() {    
    return asyncCall()    
        .then(    
            'some string',    
            [                                                                                       
                1,    
                2,    
                3    
            ]    
        );    
}  

Run:

$  ./node_modules/.bin/eslint --env es6  --no-eslintrc --rule 'indent: [2, 4]' test.js 

/home/kostya/test.js
  5:13  error  Expected indentation of 8 space characters but found 12   indent
  6:17  error  Expected indentation of 12 space characters but found 16  indent
  7:17  error  Expected indentation of 12 space characters but found 16  indent
  8:17  error  Expected indentation of 12 space characters but found 16  indent
  9:13  error  Expected indentation of 8 space characters but found 12   indent

✖ 5 problems (5 errors, 0 warnings)

What did you expect to happen?

No errors.

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

See above.

[oatkiller]

👍 Thanks for checking this out. I’m having the same issue.

[sstern6]

@platinumazure working on something i believe is related, https://github.com/eslint/eslint/issues/4161#issuecomment-247742813.

Digging into indent.js looks like eslint is having issues indenting with arrays on multiple lines. I have found a few viable ways to approach this but there are a few issues open related to this I am not sure how deep to go, and depending on the approach starts breaking multiple existing tests and each case is different. Looks like theres issues with ArrayExpressions indenting with, nested arrays, objects, and string literals from what I can see.

A few seemingly related issues:

https://github.com/eslint/eslint/issues/4161 https://github.com/eslint/eslint/issues/6007 https://github.com/eslint/eslint/issues/3737