Skip to content

Defer creation of node typings installer #52904

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

Merged
merged 3 commits into from
Feb 22, 2023
Merged

Defer creation of node typings installer #52904

merged 3 commits into from
Feb 22, 2023

Conversation

sheetalkamat
Copy link
Member

Alternative suggestion by @sandersn in #52817
Fixes #52705

Copy link
Member

@jakebailey jakebailey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sheetalkamat
Copy link
Member Author

@mjbvz @zkat with this change it also means that ti.log file wont be created unless typing installer is used. Will this break any of the error logging/telemetry or other test/log collection.

@mjbvz
Copy link
Contributor

mjbvz commented Feb 22, 2023

Not that I'm aware of on the VS Code side. I almost never look at or request users send the ti.log, just the tsserver.log

@sheetalkamat sheetalkamat merged commit 27f20c9 into main Feb 22, 2023
@sheetalkamat sheetalkamat deleted the defferedTICreate branch February 22, 2023 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Author: Team For Uncommitted Bug PR for untriaged, rejected, closed or missing bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TS Server stats files in ATA cache even if ATA is not being used
5 participants