-
Notifications
You must be signed in to change notification settings - Fork 5
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
Put the LSP implementation at Eclipse.org, with Ceylon #9
Comments
It would be fine with me for the Eclipse/Ceylon project to absorb this project, but I think the decision would be up to @gavinking and @FroMage. So far, I am the only contributor. I suspect we'd want to see more activity and interest before making this officially supported by the Ceylon project. |
I'm not opposed to it at all, but I think we should first wait for Eclipse to accept or not Ceylon. |
How to succeed? |
The Eclipse Ceylon project is now provisionned and working. So I think the migration of the LS to github.com/eclipse/ceylon would now be possible. Even the VSCode specific parts could actually be part of the Ceylon project at Eclipse.org (but it would still be better to split those 2 parts in 2 distinct contributions). |
@mickaelistria @FroMage After speaking with @jvasileff, I've already started looking into integrating his previous work on the VSCode LSP, with the However, it is in an early stage, and I didn't push anything for now, but I would expect adding it into the eclipse Ceylon GitHub repositories at some stage, under the repo: Since there's no planned date for the Eclipse Ceylon 1.4 release for now, I started this work based on the |
Thanks @davidfestal |
@mickaelistria wdyt about this question ? |
As a user or an integrator, I'm fine with it. About the legal part, I guess that Ceylon 1.3.3 was already IP-approved by Eclipse Fondation, wasn't it? Another possibility, if there is no deadline nor constraints, would be that you write the LS on top of Ceylon 1.4 snapshot, and just release this together with Ceylon 1.4. But doing this, it means you couldn't release the LS earlier. Not sure whether it would be an issue. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
I've hidden the questions/comments about debugging. Please keep comments germane to the issue. Thanks! |
The Ceylon language is in the process of moving to the Eclipse.org Foundation to ensure good governance: https://projects.eclipse.org/proposals/eclipse-ceylon . The Eclipse Foundation is also the host of the LSP4J project that is used by the implementation of Ceylon Language Server, and of several LSP related initiatives ( https://www.eclipse.org/community/eclipse_newsletter/2017/may/ ). It would make a lot of sense to move the Ceylon LSP implementation inside the Ceylon project as soon as this is moved to Eclipse.org !
The text was updated successfully, but these errors were encountered: