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

ruff server update lsp-types to 0.96.0 #11595

Open
ismail opened this issue May 29, 2024 · 7 comments
Open

ruff server update lsp-types to 0.96.0 #11595

ismail opened this issue May 29, 2024 · 7 comments
Labels
help wanted Contributions especially welcome server Related to the LSP server

Comments

@ismail
Copy link

ismail commented May 29, 2024

Looks like https://github.com/gluon-lang/lsp-types has released version 0.96.0 with notebook support, would it be possible to use it instead of using a custom fork?

@dhruvmanila
Copy link
Member

Yeah, that should be possible although @snowsignal might know more if there's any other difference apart from the Notebook support but she's on PTO for the week. Feel free to open a PR if you're interested :)

@dhruvmanila dhruvmanila added help wanted Contributions especially welcome server Related to the LSP server labels May 29, 2024
@dhruvmanila dhruvmanila changed the title Unfork lsp-types ruff server unfork lsp-types May 29, 2024
@dhruvmanila dhruvmanila changed the title ruff server unfork lsp-types ruff server update lsp-types to 0.96.0 May 29, 2024
@MichaReiser
Copy link
Member

Yeah, I think that should be possible. All we needed was that the notebook PR gets merged and released and that's now the case.

@T-256
Copy link
Contributor

T-256 commented May 31, 2024

Also, we should consider gluon-lang/lsp-types#284

@MichaReiser
Copy link
Member

Hmm, that sounds painful. I don't think we're in a rush updating. Let's see what they decide on.

@T-256
Copy link
Contributor

T-256 commented May 31, 2024

cc @karthiknadig who announced lsprotocol crate at here and was looking for feedback. I recommend you try ruff_server migrated to lsprotocol. I tried but there were few blockers that stopped me on continue it. you can catch these blockers as feedback to improve lsprotocol crate.

@snowsignal
Copy link
Contributor

The issue blocking an upgrade to 0.96.0, as @T-256 pointed out, is gluon-lang/lsp-types#284. We use the old Url type frequently in the codebase and the new Uri type lacks several key functions that we need, especially related to file path conversion.

I would be interested in exploring a move to lsprotocol, though as @T-256 said there could be blockers there too.

@dhruvmanila
Copy link
Member

I tried but there were few blockers that stopped me on continue it.

Can you enumerate the blockers here? We'd appreciate that. It could also be useful to provide the diff here and anyone from the team can enumerate the said blockers.

@snowsignal snowsignal added this to the Ruff Server: Post-Stable milestone Jun 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Contributions especially welcome server Related to the LSP server
Projects
None yet
Development

No branches or pull requests

5 participants