Skip to content

Backwards compatible CompletionImport#as_underscore#22284

Merged
ChayimFriedman2 merged 2 commits intorust-lang:masterfrom
xitep:master
May 4, 2026
Merged

Backwards compatible CompletionImport#as_underscore#22284
ChayimFriedman2 merged 2 commits intorust-lang:masterfrom
xitep:master

Conversation

@xitep
Copy link
Copy Markdown
Contributor

@xitep xitep commented May 4, 2026

Commit f42e02aa introduced a new, required field which breaks auto-import in emacs in the very same fashion as issue #18767.

This PR makes the field optional; it does not, however, fix the issue per se for emacs users since lsp-mode still sends a null.

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 4, 2026
Copy link
Copy Markdown
Contributor

@ChayimFriedman2 ChayimFriedman2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe lsp-mode should fix their code, but this change is small enough that I don't resist.

Thanks!

View changes since this review

@ChayimFriedman2 ChayimFriedman2 added this pull request to the merge queue May 4, 2026
Merged via the queue into rust-lang:master with commit 91c7c2c May 4, 2026
18 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 4, 2026
@Veykril
Copy link
Copy Markdown
Member

Veykril commented May 5, 2026

not the first time we had to fix this emacs bug on our side

xitep added a commit to xitep/lsp-mode that referenced this pull request May 5, 2026
jcs090218 pushed a commit to emacs-lsp/lsp-mode that referenced this pull request May 5, 2026
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.

4 participants