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 #17073: Better inline preview for postfix completion #17081

Merged
merged 1 commit into from
Apr 16, 2024

Conversation

davidbarsky
Copy link
Contributor

See discussion on #17077, but I strongly suspect that the changes to the TextEdit ranges caused VS Code's autocomplete to prefer the snippets over method completions. I explain why I think that here.

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 15, 2024
@Veykril
Copy link
Member

Veykril commented Apr 16, 2024

@bors r+

@bors
Copy link
Contributor

bors commented Apr 16, 2024

📌 Commit ab8c112 has been approved by Veykril

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Apr 16, 2024

⌛ Testing commit ab8c112 with merge dcbb27a...

@bors
Copy link
Contributor

bors commented Apr 16, 2024

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

@bors bors merged commit dcbb27a into rust-lang:master Apr 16, 2024
11 checks passed
lnicola pushed a commit to lnicola/rust-analyzer that referenced this pull request Apr 17, 2024
…kril

Revert rust-lang#17073: Better inline preview for postfix completion

See discussion on rust-lang#17077, but I strongly suspect that the changes to the `TextEdit` ranges caused VS Code's autocomplete to prefer the snippets over method completions. I explain why I think that [here](rust-lang#17077 (comment)).
@davidbarsky davidbarsky deleted the david/revert-17073 branch November 21, 2024 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants