-
Notifications
You must be signed in to change notification settings - Fork 9
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
Move jupyterlab-lsp
server extension into jupyter-server
organization
#30
Comments
Thank you for opening your first issue in this project! Engagement like this is essential for open source projects! 🤗 |
Sorry there hasn't been any movement on this. Let's discuss this again at our next meeting. Pinging the @jupyter-server/jupyter-server-council to review this proposal and respond. |
Can someone give me permissions to this repo so I can vote... |
👍 @ellisonbg, just sent you an invitation to join the Jupyter Server org. |
A potential downside of this proposal is that moving issues between organizations is annoying. We often bump into this for notebook/lab/server during Thursday triage meetings (and often no one has sufficient permissions to ask our friend meseeksdev to move issues), and I suspect it might be an issue with LSP which under this proposal will live in 3 organizations for some time: JupyterLab (core), jupyter-lsp (extra features), jupyter-server and users will have no clue where to report. |
@krassowski yes, this is a great point. This issue was mentioned during the Jupyter Server team meeting where this org transfer was first proposed. I think you and the rest of the LSP maintainers should definitely take some time to consider if this is the best move forward for the project. In the meantime, we can continue the vote here. That way, we will know if the server team is willing to receive the (server-side) project as something they will shepherd. If it's accepted, the LSP maintainers can still decide they don't want to do the transfer after all. |
Just to clarify, I was not saying that I oppose the move, but that it might make our lives easier if we have that repo in Some other points:
I am really looking forward to moving the server extension out of the current monorepo at https://github.com/jupyter-lsp/jupyterlab-lsp - it would simplify moving to jupyter-releaser, dealing with changelogs and also make first contributions easier. I think I prefer if that would be under |
Hi @krassowski, thanks for your clarification. I have no objection to keeping the |
It seems there is agreement about keeping the jupyter-lsp server extension in the jupyter-lsp organization. |
Closing as the discussion seems to have concluded |
There is an ongoing effort to add the support of the Language Server Protocol into core JupyterLab.
Michał Krassowski opened a JEP (#72) proposing the adoption of LSP as part of Jupyter for notebooks. This was approved and merged in September 2021.
Most of the work on LSP in Jupyter lives in the jupyterlab-lsp project, mostly maintained by Michał Krassowski (@krassowski) and Nick Bollweg (@bollwyvl ).
I have been working on integrating some foundational parts of the logic of
jupyterlab-lsp
in core JupyterLab. In PR Integratejupyterlab-lsp
intojupyterlab
jupyterlab/jupyterlab#12534, core components of thejupyterlab-lsp
frontend extension handling the language server initialization and communication are moved into a new "lsp" package of JupyterLab.This work has already been extensively discussed in dedicated meetings with the developers of
jupyterlab-lsp
A natural next step would be to create a new repository in jupyter-server incorporating the pure server-side parts of
jupyterlab-lsp
(jupyter-lsp), and make it a dependency ofjupyterlab-server
(Addjupyter-lsp
to dependencies list jupyterlab/jupyterlab_server#282).For reference, a JupyterLab team compass issue is opened here on the same subject: jupyterlab/frontends-team-compass#148.
Ping @bollwyvl and @krassowski, please feel free to comment or correct me if I misrepresented anything.
Votes
The text was updated successfully, but these errors were encountered: