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

bashls triggers a registerCapability handler despite dynamicRegistration set to false #722

Closed
WhyNotHugo opened this issue Feb 8, 2023 · 8 comments
Labels
bug Something isn't working priority ⭐️ Triaged and deemed a priority

Comments

@WhyNotHugo
Copy link

Code editor

neovim

Platform

Linux

Version

4.6.2

What steps will reproduce the bug?

Just run neovim with the LSP configured.

How often does it reproduce? Is there a required condition?

Always, at startup.

What is the expected behavior?

No response

What do you see instead?

Neovim's logs state:

[WARN][2023-02-08 18:16:51] ...lsp/handlers.lua:113	"The language server bashls triggers a registerCapability handler despite dynamicRegistration set to false. Report upstream, this warning is harmless"

Additional information

Nothing special about my setup. I don't see any obvious issues.

@skovhus skovhus added the bug Something isn't working label Feb 8, 2023
@skovhus
Copy link
Collaborator

skovhus commented Feb 8, 2023

Thanks for reporting this. Wondering why we never saw this issue before, maybe users are on an old version?

@skovhus skovhus added the priority ⭐️ Triaged and deemed a priority label Feb 8, 2023
@WhyNotHugo
Copy link
Author

WhyNotHugo commented Feb 8, 2023 via email

@David-Else
Copy link
Contributor

I have noticed that in Helix I am able to use the rename code action, and only when I press enter it tells me 'language server does not support symbol renaming'. Could this be related, I think other servers warn in advance, but I might be misremembering.

@skovhus
Copy link
Collaborator

skovhus commented Mar 9, 2023

Closed by #763

let me know if the new version works for you.

@skovhus skovhus closed this as completed Mar 9, 2023
@David-Else
Copy link
Contributor

I have noticed that in Helix I am able to use the rename code action, and only when I press enter it tells me 'language server does not support symbol renaming'. Could this be related, I think other servers warn in advance, but I might be misremembering.

This still happens with 4.8.3, should it report it does not support renaming before prompting to enter the new name?

@skovhus
Copy link
Collaborator

skovhus commented Mar 9, 2023

So the server does not support renaming, so I’m curious why helix believes this. Is this the same for other languages and LSP severs?

@David-Else
Copy link
Contributor

I don't know any other servers that don't support renaming, so I am not sure how to test. Maybe @the-mikedavis can shed some light on it.

@the-mikedavis
Copy link
Contributor

That's an issue on our end - we check the server capabilities after entering the replacement in the prompt. I opened up helix-editor/helix#6249 to improve that UX

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working priority ⭐️ Triaged and deemed a priority
Projects
None yet
Development

No branches or pull requests

4 participants