-
Notifications
You must be signed in to change notification settings - Fork 12.6k
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
Only when typings file change for the project, schedule the update for the project #42428
Conversation
@typescript-bot pack this |
Heya @sheetalkamat, I've started to run the tarball bundle task on this PR at 6d543ea. You can monitor the build here. |
Hey @sheetalkamat, I've packed this into an installable tgz. You can install it for testing by referencing it in your
and then running There is also a playground for this build and an npm module you can use via |
Working, thanks so much! |
Are there any scenarios we should expect to be affected in VS besides adding new typings? I don't foresee any issues in that case. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very nice
No it just improves when the program updates in background in tsserver so technically nothing should change apart from better perf with respect to avoiding unnecessary updates. |
This avoids unnecessary project updates, project updated in background on which editor normally sends error request so multiple program updates and unnecessarily invalidating resolutions is avoided.
Fixes #39326