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

fix: Fix completions disappearing when typing two keys in quick succession #12431

Merged
merged 2 commits into from
Jun 1, 2022

Conversation

Veykril
Copy link
Member

@Veykril Veykril commented May 31, 2022

With this PR we now retry requests if they get cancelled due to document changes.

This fixes the completions problem we have where completions seem to randomly disappear, see https://rust-lang.zulipchat.com/#narrow/stream/185405-t-compiler.2Frust-analyzer/topic/Completions.20not.20always.20appearing

Fixes #10187
Fixes #7560
Fixes #12153

@Veykril Veykril changed the title fix: Fix completions disappearing on every quick odd keystroke fix: Fix completions disappearing when typing two keys in quick succession May 31, 2022
@Veykril Veykril force-pushed the request-retry branch 2 times, most recently from d03b6af to 3354988 Compare May 31, 2022 22:17
@Veykril Veykril marked this pull request as ready for review May 31, 2022 22:23
@Veykril
Copy link
Member Author

Veykril commented May 31, 2022

r? @matklad

@Veykril
Copy link
Member Author

Veykril commented Jun 1, 2022

@bors r+

@bors
Copy link
Contributor

bors commented Jun 1, 2022

📌 Commit d88ae66 has been approved by Veykril

@bors
Copy link
Contributor

bors commented Jun 1, 2022

⌛ Testing commit d88ae66 with merge 4f5c7aa...

@bors
Copy link
Contributor

bors commented Jun 1, 2022

☀️ Test successful - checks-actions
Approved by: Veykril
Pushing 4f5c7aa to master...

@bors bors merged commit 4f5c7aa into rust-lang:master Jun 1, 2022
@Veykril Veykril deleted the request-retry branch June 1, 2022 10:54
@0x5c
Copy link

0x5c commented Jun 1, 2022

I'm not familiar with the r-a release cycle, how long might this take to land in the VSCode extension?

@Veykril
Copy link
Member Author

Veykril commented Jun 1, 2022

next monday for the stable release, tomorrow (~4hrs) for the nightly one

@lnicola
Copy link
Member

lnicola commented Jun 1, 2022

@0x5c five days, or you can install a pre-release version tomorrow

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants