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

Move jupyterlab-lsp server extension into jupyter-server organization #30

Closed
5 of 28 tasks
trungleduc opened this issue Jun 9, 2022 · 10 comments
Closed
5 of 28 tasks
Labels
enhancement New feature or request

Comments

@trungleduc
Copy link

trungleduc commented Jun 9, 2022

There is an ongoing effort to add the support of the Language Server Protocol into core JupyterLab.

  1. 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.

  2. Most of the work on LSP in Jupyter lives in the jupyterlab-lsp project, mostly maintained by Michał Krassowski (@krassowski) and Nick Bollweg (@bollwyvl ).

  3. I have been working on integrating some foundational parts of the logic of jupyterlab-lsp in core JupyterLab. In PR Integrate jupyterlab-lsp into jupyterlab jupyterlab/jupyterlab#12534, core components of the jupyterlab-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

  4. 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 of jupyterlab-server (Add jupyter-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

@trungleduc trungleduc added the enhancement New feature or request label Jun 9, 2022
@welcome
Copy link

welcome bot commented Jun 9, 2022

Thank you for opening your first issue in this project! Engagement like this is essential for open source projects! 🤗

If you haven't done so already, check out Jupyter's Code of Conduct. Also, please try to follow the issue template as it helps other other community members to contribute more effectively.
welcome
You can meet the other Jovyans by joining our Discourse forum. There is also an intro thread there where you can stop by and say Hi! 👋

Welcome to the Jupyter community! 🎉

@Zsailer
Copy link
Member

Zsailer commented Aug 29, 2022

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.

@ellisonbg
Copy link

Can someone give me permissions to this repo so I can vote...

@Zsailer
Copy link
Member

Zsailer commented Aug 29, 2022

👍 @ellisonbg, just sent you an invitation to join the Jupyter Server org.

@krassowski
Copy link

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.

@Zsailer
Copy link
Member

Zsailer commented Aug 30, 2022

@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.

@krassowski
Copy link

krassowski commented Sep 3, 2022

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 jupyter-lsp organisation.

Some other points:

  • If I were a new contributor I would expect to find jupyter-lsp server extension in jupyter-lsp organization
  • on the point about issues: how frequently do we expect to move issues between jupyterlab-lsp and jupyter-lsp and how frequently between jupyter-lsp and jupyter-server (or other repos in this org)?
  • maybe we should rename jupyter-lsp repo to jupyter-lsp-server?
  • we can add designated members (or everyone) from jupyter-server team who would need access to the jupyter-lsp organization and create a GitHub team with privileges to that repo so that everyone can triage/release as appropriate

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 https://github.com/jupyter-lsp organization, but I don't feel strongly about it. @trungleduc what do you think? If you are available, let's aim to get this done by the end of next week.

@trungleduc
Copy link
Author

Hi @krassowski, thanks for your clarification. I have no objection to keeping the jupyter-lsp server extension in the jupyter-lsp organization.
Since the lsp related development will continue to happen on both jupyterlab and jupyter-lsp org for a period, I think we can postpone the transfer to the jupyter-server org. And as you said, I also want to move the server extension outside the monorepo jupyterlab-lsp.
I'm on other stuff this week but I can get back to it next week.

@damianavila
Copy link

Hi @krassowski, thanks for your clarification. I have no objection to keeping the jupyter-lsp server extension in the jupyter-lsp organization.

It seems there is agreement about keeping the jupyter-lsp server extension in the jupyter-lsp organization.
Then, should we close this one?

@vidartf
Copy link
Member

vidartf commented Oct 14, 2022

Closing as the discussion seems to have concluded

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

6 participants