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

LSP: Fix codeAction/resolve server capability check #8421

Merged
merged 1 commit into from
Oct 2, 2023

Conversation

the-mikedavis
Copy link
Member

Previously we accidentally checked the server's completion resolve capability rather than the code action resolve capability. It looks like this was an oversight introduced in #7677.

Fixes #8413
Fixes astral-sh/ruff-lsp#249

Previously we accidentally checked the server's _completion_ resolve
capability rather than the code action resolve capability.
@the-mikedavis the-mikedavis added C-bug Category: This is a bug E-easy Call for participation: Experience needed to fix: Easy / not much A-language-server Area: Language server client S-waiting-on-review Status: Awaiting review from a maintainer. labels Sep 29, 2023
Copy link
Member

@pascalkuthe pascalkuthe left a comment

Choose a reason for hiding this comment

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

great catch!

@pascalkuthe pascalkuthe added this to the 23.9 milestone Sep 29, 2023
@archseer archseer merged commit 6abaf3d into master Oct 2, 2023
6 checks passed
@archseer archseer deleted the lsp-fix-code-action-resolve-capabilities-check branch October 2, 2023 03:32
danillos pushed a commit to danillos/helix that referenced this pull request Nov 21, 2023
Previously we accidentally checked the server's _completion_ resolve
capability rather than the code action resolve capability.
dgkf pushed a commit to dgkf/helix that referenced this pull request Jan 30, 2024
Previously we accidentally checked the server's _completion_ resolve
capability rather than the code action resolve capability.
mtoohey31 pushed a commit to mtoohey31/helix that referenced this pull request Jun 2, 2024
Previously we accidentally checked the server's _completion_ resolve
capability rather than the code action resolve capability.
smortime pushed a commit to smortime/helix that referenced this pull request Jul 10, 2024
Previously we accidentally checked the server's _completion_ resolve
capability rather than the code action resolve capability.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-language-server Area: Language server client C-bug Category: This is a bug E-easy Call for participation: Experience needed to fix: Easy / not much S-waiting-on-review Status: Awaiting review from a maintainer.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Always shown ruff-lsp code actions do nothing Organize Imports and Fix All do nothing in helix
3 participants