Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(provider): read TS Lint indent size #134

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mattyclarkson
Copy link

The TS Lint indent rule can specify the indentation size:

"indent": [true, "spaces"]
"indent": [true, "spaces", 4]
"indent": [true, "tabs", 2]

This patch allows tsfmt to read that size and correctly set the formatting options. This means that tslint and tsfmt do not fight over the indentation size.

The TS Lint [`indent` rule](https://palantir.github.io/tslint/rules/indent/) can specify the indentation size:

```
"indent": [true, "spaces"]
```
```
"indent": [true, "spaces", 4]
```
```
"indent": [true, "tabs", 2]
```

This patch allows `tsfmt` to read that size and correctly set the formatting options. This means that `tslint` and `tsfmt` do not fight over the indentation size.
@mattyclarkson
Copy link
Author

mattyclarkson commented Sep 27, 2018

@vvakame not sure if the travis error is a result of this patch?

@mattyclarkson
Copy link
Author

@vvakame any chance you could have a look at this? Would be a really good quality of life improvement for the project; means that with a tslint.json there is zero need for a tsfmt.json if there is a change in indent specified.

I'm not sure why console is being defined globally twice on the CI.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant