Skip to content

fix: don't complete .await on receivers of unknown type#22486

Merged
Veykril merged 3 commits into
rust-lang:masterfrom
ada4a:22478-spurious-await
Jun 19, 2026
Merged

fix: don't complete .await on receivers of unknown type#22486
Veykril merged 3 commits into
rust-lang:masterfrom
ada4a:22478-spurious-await

Conversation

@ada4a

@ada4a ada4a commented May 29, 2026

Copy link
Copy Markdown
Contributor

Fixes #22478

This also happens to improve syntax highlighting in presence of error types, and therefore fixes #22487

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 29, 2026
@ada4a ada4a marked this pull request as draft May 29, 2026 16:25
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 29, 2026
Comment thread crates/hir-ty/src/traits.rs Outdated
Comment thread crates/hir/src/lib.rs Outdated
Comment thread crates/ide-completion/src/completions/dot.rs Outdated
@ada4a ada4a force-pushed the 22478-spurious-await branch from 21a8536 to fd4b9ff Compare June 1, 2026 19:26
@ada4a ada4a marked this pull request as ready for review June 1, 2026 19:26
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jun 1, 2026
@ada4a ada4a force-pushed the 22478-spurious-await branch 2 times, most recently from dac2dfe to 0f3c0ec Compare June 1, 2026 19:40
@rustbot

This comment has been minimized.

@rustbot

This comment has been minimized.

Comment thread crates/ide-completion/src/completions/dot.rs Outdated
@ada4a ada4a force-pushed the 22478-spurious-await branch from 0f3c0ec to 52ecb67 Compare June 17, 2026 11:24
@rustbot

rustbot commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator

This PR was rebased onto a different master commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

@ada4a ada4a force-pushed the 22478-spurious-await branch from 52ecb67 to 6037856 Compare June 17, 2026 11:53
@Veykril Veykril added this pull request to the merge queue Jun 19, 2026
Merged via the queue into rust-lang:master with commit 6b6c7c0 Jun 19, 2026
18 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jun 19, 2026
@ada4a ada4a deleted the 22478-spurious-await branch June 19, 2026 08:25
@ada4a

ada4a commented Jun 19, 2026

Copy link
Copy Markdown
Contributor Author

I was actually waiting for a response to #22486 (comment), but I can do that as a separate PR if needed

@Veykril

Veykril commented Jun 19, 2026

Copy link
Copy Markdown
Member

ah my bad. But yea we can address that if Chayim finds t he time look over that.

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.

callable token modifier emitted on locals of unknown type .await completion shown in a non-async function when the receiver is of unknown type

4 participants