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

Remove linux commands, respect tls_skip_insecure as completionParam and set correct arch on amd64 #17

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

badbyte7
Copy link

I changed the runCommand part to kotlin/java code, so that the llm-ls download also works under windows, and if curl is not installed (newer Ubuntu verions). It also respects proxy and certificate settings.

In my two tested Ubuntu versions the architecture was amd64, where no llm-ls is available. I added a hardcoded conversion to x86_64.

I also realised, that the llm-ls has a tls_skip_verify_insecure CompletionParameter prepared and the plugin also has the setting, but it was not sent in the completion request to the llm-ls. This setting/parameter will be sent now.

The only thing still open are the proxy settings for the llm-ls. I actually set the https_proxy environment variable before I launch the Idea, but it would be much nicer, if the plugin could do this to. But up to now I could not read the proxy settings of the IDE.

I hope this helps and the quality of the code is ok. ;-)

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.

2 participants