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

Provide an option to hide deprecated items from completion #18006

Merged
merged 1 commit into from
Sep 1, 2024

Conversation

ChayimFriedman2
Copy link
Contributor

Fixes #17989.

I wonder if this should be instead done in the editor, that will do it in a language-agnostic way. Can't hurt to do it in rust-analyzer, I guess.

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

Veykril commented Aug 30, 2024

This would be better as an editor config, but people complain when I say that (because at least VSCode never does anything) so in r-a it goes :/

@Veykril
Copy link
Member

Veykril commented Aug 30, 2024

This will likely conflict with #17945 and that PR has more issues with rebasing so I'll wait on that

@bors
Copy link
Collaborator

bors commented Aug 31, 2024

☔ The latest upstream changes (presumably #17945) made this pull request unmergeable. Please resolve the merge conflicts.

@Veykril
Copy link
Member

Veykril commented Sep 1, 2024

@bors r+

@bors
Copy link
Collaborator

bors commented Sep 1, 2024

📌 Commit 01cc847 has been approved by Veykril

It is now in the queue for this repository.

@bors
Copy link
Collaborator

bors commented Sep 1, 2024

⌛ Testing commit 01cc847 with merge 715e9ff...

@bors
Copy link
Collaborator

bors commented Sep 1, 2024

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

@bors bors merged commit 715e9ff into rust-lang:master Sep 1, 2024
11 checks passed
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.

Add option to hide deprecated methods from completion results
4 participants