-
Notifications
You must be signed in to change notification settings - Fork 27
Use LSP instead of legacy --ide-* parameters #207
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
Conversation
wow! thanks for this! I'd like to have a deprecation period and a fallback to the traditional ide commands set via configuration. meaning, it would be nice to default to your new code but have an option in the settings to switch back to ide-* mode. do you think that's possible? |
Maybe? I don't know... Having some sort of a switch in the configuration, etc... I think we have two simpler options:
The last time it updated was almost a year ago, so it is not like someone would be missing frequently coming updates etc.
I think that any of these is simpler than having a mixed codebase with switching logic etc. |
My fear that there will be funky bugs and no one to support the changes. If you want to sign up to support bugs for a little while, let's go ahead and just land it and publish it after the nushell release (which is supposed to be today). |
I am fine with supporting bugs on the plugin side (because there is little-to-no code to support, it is all LSP ;) ) But to play it very safe, let's publish a pre-release version of the plugin. People can opt-in for the new version and with one click opt-out if something goes wrong. This is literally how every other plugin does it, and how VSCode plugins life cycle is designed to be ![]() We wait for a month or so, watch the feedback, and then make it as a main release. Do you agree @fdncred ? |
Sounds good but I'm not sure how to make it pre-release. |
Docs say it is fairly easy, just by adding vsce package --pre-release
vsce publish --pre-release Ref: https://code.visualstudio.com/api/working-with-extensions/publishing-extension#prerelease-extensions |
ok, thanks. i can try that. we just released nushell 0.107.0 so it's a good time for me to try and remember how to do this release. |
@AlexeyRaga I'm trying to test the latest main branch on Windows and am getting all kinds of errors. Any ideas?
|
I updated a bunch of deps and now am getting this, along with the other errors.
If I run nu --lsp from the repl i get this. it looks like it's trying to send those prints via stdout
|
@AlexeyRaga The new version is published but I don't see the If I uninstall, I can then install the pre-release version though. |
@fdncred I do have it, without having to uninstall etc: Maybe it is some caching, or a VSCode required to restart, I don't know... But worked for me |
ok, glad it worked for someone. thanks for letting me know. |
Description
Switch from using specific
--ide-*
legacy parameters to using a full LSP server (provided bynu --lsp
).In this mode the plugin will automatically detect capabilities that LSP provides and will enable/support appropriate feature set in the IDE.
In particular, renaming capability works in this mode, which addresses #206 and, likely, delivers other goodness.