Code style summary to discuss and configure

[arogachev]

Trailing comma after last argument (if they are split into multiple lines):

$this->assertSame(
    ['' => ['Value must be no less than 5.']],
    $result->getErrorMessagesIndexedByPath(),
);

vs

$this->assertSame(
    ['' => ['Value must be no less than 5.']],
    $result->getErrorMessagesIndexedByPath()
);

The first option is already used for array elements, but not automated.

[roxblnfk]

I like the first. The same about a function parameters declaration.

[tomaszkane]

First one. It's more consistent with arrays:

$array = [
'foo',
'boo',
];

and it easier/more friendly to developer - needs less actions when move last item:

1. move line

vs

1. move line,
2. add comma to moved line
3. remove comma to actual last line

[samdark]

1st way is good to go.

[arogachev]

https://docs.styleci.io/fixers#trailing_comma_in_multiline_call