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

TSLint is really slow #1731

Closed
benlesh opened this issue May 24, 2016 · 8 comments
Closed

TSLint is really slow #1731

benlesh opened this issue May 24, 2016 · 8 comments

Comments

@benlesh
Copy link
Member

benlesh commented May 24, 2016

Do we have any other alternatives to TSLint?

cc/ @alexeagle @robwormald @david-driscoll

@david-driscoll
Copy link
Member

What part feels slow for you? I've never had large issues with perf on TSLint itself, but that's just my experience.

Does your editor have a TSLint plugin to help linting warning as you go?
(without knowing the setup it's hard to say!)

@alexeagle
Copy link
Contributor

There is discussion going on about a replacement that's based on TS language services (tslint can't do cross-file program analysis at present because it can't boot up a typechecker)
Do you have a link on your CI or somewhere showing the timing?

@trxcllnt
Copy link
Member

@alexeagle our current travis config runs npm install && npm run lint together, so we don't have the linter timing isolated. Together they took ~125s in the most recent build.

@kwonoj
Copy link
Member

kwonoj commented Jun 15, 2016

linking palantir/tslint#1131 for reference.

kwonoj added a commit to kwonoj/rxjs that referenced this issue Jul 1, 2016
kwonoj added a commit to kwonoj/rxjs that referenced this issue Jul 1, 2016
kwonoj added a commit to kwonoj/rxjs that referenced this issue Jul 8, 2016
kwonoj added a commit to kwonoj/rxjs that referenced this issue Jul 8, 2016
kwonoj added a commit to kwonoj/rxjs that referenced this issue Jul 8, 2016
kwonoj added a commit to kwonoj/rxjs that referenced this issue Jul 9, 2016
@kwonoj
Copy link
Member

kwonoj commented Aug 26, 2016

just fyi, maybe https://github.com/eslint/typescript-eslint-parser this can shed light in further. (long, long time later since it's experimental for now..)

@prabirshrestha
Copy link

When Lint extensions land in typescript it could solve the perf problem. microsoft/TypeScript#10162

@kwonoj
Copy link
Member

kwonoj commented Jun 29, 2017

This one is not immediate concern anymore by lint-stage hooks and we can improve over once we have better way, closing.

@kwonoj kwonoj closed this as completed Jun 29, 2017
@lock
Copy link

lock bot commented Jun 6, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Jun 6, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants