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

Translate TSLint max-line-length into Prettier printWidth automatically #25

Open
aleclarson opened this issue Dec 10, 2018 · 1 comment

Comments

@aleclarson
Copy link
Collaborator

aleclarson commented Dec 10, 2018

Prettier's printWidth defaults to 80, which (if left unchanged) makes TSLint's max-line-length look broken. The reality is that users should be using Prettier options to control the maximum line length.

To ease the pain, this package could probably translate TSLint's max-line-length into Prettier's printWidth automatically when printWidth is not explicitly defined.

We could also warn when both are defined, but have different values.

Related: #15

@sveyret
Copy link

sveyret commented Dec 12, 2018

Well, probably the initial #15 issue was not correctly written, but this is exactly what was expected (I mean, set the printWidth with max-line-length value). Wouldn't it be better to track this issue with the initial issue, as many people added comments which may be useful for dev ?

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

No branches or pull requests

2 participants