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

Upgrade TSLint & TypeScript, disable formatting rules #251

Merged
merged 3 commits into from
Apr 21, 2020

Conversation

adidahiya
Copy link
Contributor

No description provided.

@adidahiya adidahiya merged commit dac1dbc into master Apr 21, 2020
@adidahiya adidahiya deleted the ad/upgrade-tslint branch April 21, 2020 23:05
@SaeedZhiany
Copy link

@adidahiya

I'm wondering why the formatting rules have been disabled? are all they covered by tslint?

is it safe to upgrade my project tslint and tslint-react to 6.1.2 and 5.0.0, respectively (I'm currently using tslint 6.1.1 and tslint 4.2.0)? (I know we should migrate to eslint and prettier, but I still want to use them until I found some free times to migration)

Thanks

@adidahiya
Copy link
Contributor Author

They're removed because we recommend using prettier. You don't have to migrate to eslint to do this, it's pretty easy with tslint-plugin-prettier. The change to remove formatting rules from tslint and tslint-react obviates tslint-config-prettier. Related PR: palantir/tslint#4871

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.

2 participants