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

Revert "Optimistically show paste with imports if TS server takes to long when computing imports to add" #241198

Closed
wants to merge 1 commit into from

Conversation

aiday-mar
Copy link
Contributor

@aiday-mar aiday-mar commented Feb 19, 2025

Reverts #239899

cc @mjbvz

@aiday-mar aiday-mar enabled auto-merge (squash) February 19, 2025 12:01
@vs-code-engineering vs-code-engineering bot added this to the February 2025 milestone Feb 19, 2025
@aiday-mar aiday-mar disabled auto-merge February 19, 2025 12:11
@mjbvz
Copy link
Collaborator

mjbvz commented Feb 19, 2025

Was the change causing any issues? It fixes a important case where the spinner could show

@mjbvz
Copy link
Collaborator

mjbvz commented Feb 19, 2025

Ok seeing #241164 (comment). Let's fix that case instead of reverting. I suspect it could have happened before too with different timings

@aiday-mar
Copy link
Contributor Author

Hi thanks then I will close this PR

@aiday-mar aiday-mar closed this Feb 20, 2025
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.

3 participants