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

Backport of rust-analyzer "Use completion item indices instead of property matching" #132987

Merged
merged 2 commits into from
Nov 16, 2024

Conversation

lnicola
Copy link
Member

@lnicola lnicola commented Nov 13, 2024

CC rust-lang/rust-analyzer#18503
CC @SomeoneToIgnore

This fixes a problem that occurs on less-conforming LSP clients like Kate and Emacs lsp-mode.

r? @traviscross

@rustbot
Copy link
Collaborator

rustbot commented Nov 13, 2024

⚠️ Warning ⚠️

  • Pull requests are usually filed against the master branch for this repo, but this one is against beta. Please double check that you specified the right target!

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Nov 13, 2024
@rustbot
Copy link
Collaborator

rustbot commented Nov 13, 2024

rust-analyzer is developed in its own repository. If possible, consider making this change to rust-lang/rust-analyzer instead.

cc @rust-lang/rust-analyzer

@workingjubilee
Copy link
Member

cc ... @cuviper?

@jieyouxu
Copy link
Member

This probably needs T-release signoff... r? release

@rustbot rustbot added the T-release Relevant to the release subteam, which will review and decide on the PR/issue. label Nov 13, 2024
@rustbot rustbot assigned tmandry and unassigned traviscross Nov 13, 2024
@cuviper
Copy link
Member

cuviper commented Nov 13, 2024

It's fine with me if the team approves, which I assume so since @lnicola is a member. Feel free to r=me.

Usually we'd go through nominated/accepted labels and then let the release team backport it ourselves, but we don't really have a way to do that labeling for subtree projects.

@cuviper cuviper mentioned this pull request Nov 15, 2024
@cuviper cuviper assigned cuviper and unassigned tmandry Nov 15, 2024
@cuviper
Copy link
Member

cuviper commented Nov 15, 2024

I've included this in #133077.

@rustbot label +beta-accepted

@rustbot rustbot added the beta-accepted Accepted for backporting to the compiler in the beta channel. label Nov 15, 2024
@cuviper cuviper added this to the 1.83.0 milestone Nov 15, 2024
bors added a commit to rust-lang-ci/rust that referenced this pull request Nov 15, 2024
[beta] backports

- Use completion item indices instead of property matching rust-lang#132987, rust-lang/rust-analyzer#18503
- Reject raw lifetime followed by `'`, like regular lifetimes do rust-lang#132341
- Only disable cache if predicate has opaques within it rust-lang#132625
- rustdoc-search: case-sensitive only when capitals are used rust-lang#133043

r? cuviper
bors added a commit to rust-lang-ci/rust that referenced this pull request Nov 16, 2024
[beta] backports

- Use completion item indices instead of property matching rust-lang#132987, rust-lang/rust-analyzer#18503
- Reject raw lifetime followed by `'`, like regular lifetimes do rust-lang#132341
- Only disable cache if predicate has opaques within it rust-lang#132625
- rustdoc-search: case-sensitive only when capitals are used rust-lang#133043
- (ci) Update macOS Xcode to 15 rust-lang#131570

r? cuviper
@bors bors merged commit f0db79f into rust-lang:beta Nov 16, 2024
6 checks passed
@lnicola lnicola deleted the fix-completion-resolve branch December 11, 2024 09:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
beta-accepted Accepted for backporting to the compiler in the beta channel. S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-release Relevant to the release subteam, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants