jsx-wrap-multiline issue with trailing-comma
SimonSchick opened this issue · comments
Simon Schick commented
Bug Report
- tslint-react version: 4.0.0
- tslint version: 5.14.0
- typescript version: 3.4
- Running TSLint via: NodeJS v11.13
TypeScript code being linted
group.push(
<Button
id={`assign${this.props.dealer.id}`}
key='assign'
onClick={() => this.props.openAssignTable()}
>
Assign Table
</Button>,
);
with tslint.json
configuration:
"trailing-comma": [true, {
"esSpecCompliant": true,
"multiline": "always",
"singleline": "never"
}],
"jsx-wrap-multiline": true
Actual behavior
Lints fine.
Expected behavior
When a trailing comma is added the rule complaints about not being wrapped.
While this is technically correct I think an exception should be made in the case of being the single argument with a trailing comma.
Adi Dahiya commented
There are a lot of issues with this rule, see #79. My suggestion is to use prettier.
Simon Schick commented
Cool, I already disabled it, kinda sad tho.