-
Notifications
You must be signed in to change notification settings - Fork 115
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
Feature Request: Support for language server protocol #127
Comments
Thanks to this plugin: https://packagecontrol.io/packages/LSP, SublimeText could benefit from supporting the protocol as well |
As @lumnn says, such a plugin already exists for SublimeText, so I don't see the point of including LSP in PHP Companion. |
@erichard I think you misunderstood me. My point is that if the SublimePHPCompanion is a LSP compatible tool, then can be used by more editors including Sublime. On the other hand I understand it probably means a lot of work. |
@lumnn You are right, I totally misunderstood you 😄 So has I understand, LSP is all about standardizing autocompletion, go to definition, or documentation on hover. PHP Companion does nothing of these things. I see it as a smart snippet plugin, it write code in your file when you hit some specific key. I will dig deeper in the LSP spec to see if I missed something. In the meantine can you develop what you have in mind about a LSP integration ? |
@erichard I believe the I believe almost all commands could be developed as LSP code action (except goto_definition_scope which looks like |
This is just a feature request/question of how much work it would take to support the language server protocol.
Description from https://microsoft.github.io/language-server-protocol:
VS Code supports language server protocol. Since there is no good php extension today for VS Code, and if it's possible to modify SublimePHPCompanion it would make an excellent choice.
Since my python skills are close to none I cannot help with development but am wondering if this would be even possible with this library?
The text was updated successfully, but these errors were encountered: