-
Notifications
You must be signed in to change notification settings - Fork 52
Closed
Description
Unfortunately the current sample tslint.json
for tslint
incorrectly suggests the format for the indent rule is:
[true, 4]
However, the code that parses the setting expects the format to be:
[true, "spaces"]
There isn't even an option for a number in the final position (even though that seems quite sensible).
I've raised a pull request to fix the sample.
However this presents a problem for tsfmt
, specifically here.
This means:
- with a setting of
[true, 4]
,tsfmt
correct indents code buttslint
fails to apply the indent linting rule - with a setting of
[true, "spaces"]
,tsfmt
removes all indenting buttslint
correctly applies the indent linting rule
My suggestion would be that we drop the parsing of the indent rule from tslint.json
.
Metadata
Metadata
Assignees
Labels
No labels