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

Enabling Intelephense #864

Closed
JoyceBabu opened this issue Jan 25, 2020 · 3 comments
Closed

Enabling Intelephense #864

JoyceBabu opened this issue Jan 25, 2020 · 3 comments

Comments

@JoyceBabu
Copy link

I see two options for integrating with Intelephense.

Documentation for LSP-intelephense says

Install LSP and LSP-intelephense from Package Control.

This enables, LSP-Intelephense on the status bar.

The Getting Started section of this project says

Enable the server by running LSP: Enable Language Server from the command palette.

When selecting the option LSP: Enable Language Server Globally, I don't see LSP-intelephense. Instead intelephense-ls is listed. Selecting intelephense-ls adds intelephense-ls to my status bar.

Why are there two different options? Which is the official way for enabling intelephense server?

@jwortmann
Copy link
Member

Both options should work. The intelephense-ls entry has probably been there to apply default configurations for the server, when the LSP-intelephense package didn't exist yet. I would say there is no "official way", but LSP-intelephense might be the preferred option here, because it can automatically install and update the server (I think) through Package Control. (I don't use that server myself though.)

@jfcherng
Copy link
Contributor

Another reason to use LSP-intelephense is you can use variables in its initializationOptions.

#709 (comment)

@JoyceBabu
Copy link
Author

Thank you for the clarification.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants