Rule Enhancement Proposal: indent and multiline parameters.

From requireAlignedMultilineParams.

JSCS has the rule to align parameters vertically. I think this is a responsibility of indent rule.

{
    "indent": ["error", 4, {
        "FunctionDeclaration": {
            "parameters": "first", // or an integer
            "body": 1
        },
        "FunctionExpression": {
            "parameters": "first", // or an integer
            "body": 1
        }
    }]
}

Those options are similar to existing SwitchCase and VariableDeclarator options. But "parameters": "first" is special. If "parameters": "first" is specified, 2nd line of parameters and later will be aligned with the same column as the first parameter item.

/*eslint indent: ["error", 4, {"FunctionDeclaration": {"parameters": "first"}}]*/

function foo(aaa, bbb,
             ccc, ddd) {
    doSomething();
}

[BYK]

@eslint/doctrine-team we need three 👍s and a champion right? I can champion this and giving my 👍. 2 more to go I guess?

[not-an-aardvark]

The default behavior should be current behavior (backward compatibility).

It looks like with the current behavior, the indentation of function parameters isn’t checked at all:

// This is valid

/* eslint indent: [2, 2] */
function foo(
bar,
    baz,
        qux) {
  qux();
}

So we should only check parameter indentation if the user explicitly includes the parameters option.