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

[code] Add a config option to enable the developing language servers. #29319

Merged

Conversation

movie-travel-code
Copy link

The key point of the disable logic for the developing language servers is that we use another global variable, i.e. LanguageServersDeveloping, to store the language servers under developing.

If you want to enable the developing language servers, you can add the enable options in kibana.yml, like xpack.code.enableDevelopingLangServers: ['go']. And we will move the corresponding language servers into the LanguageServers array.

@elasticmachine
Copy link
Contributor

💔 Build Failed

Copy link
Contributor

@spacedragon spacedragon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why don't we enable go language server in dev mode?

server.config().get('env.dev')===true

@movie-travel-code
Copy link
Author

Why don't we enable go language server in dev mode?

server.config().get('env.dev')===true

That makes sense, thanks! For now, we can enable the developing language servers only in development mode.

@elasticmachine
Copy link
Contributor

💔 Build Failed

@movie-travel-code
Copy link
Author

@zfy0701 ping.

Define a global variable, `LanguageServersDeveloping[]`, to store the
developing language servers. And push them into `LanguageServers[]`
in development mode by default.
@elasticmachine
Copy link
Contributor

💔 Build Failed

@zfy0701
Copy link
Contributor

zfy0701 commented Jan 29, 2019

nice!

@movie-travel-code
Copy link
Author

jenkins, test this

@elasticmachine
Copy link
Contributor

💔 Build Failed

@movie-travel-code movie-travel-code merged commit f9c93be into elastic:feature/merge-code Jan 30, 2019
@movie-travel-code movie-travel-code deleted the DisableGo branch January 30, 2019 06:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants