search

sublimelsp / LSP-rust-analyzer

Convenience package for rust-analyzer
MIT License
70 stars 10 forks source link

LSP-rust-analyzer asks to install Terminus package even though it is already installed #129

Closed opedro-c closed 1 month ago

opedro-c commented 7 months ago

Hi, I'm trying to run my code through the run button placed on the main function declaration line. However, when I click on it, I'm asked to install the Terminus plugin, but when I verified it was already installed. If you need anymore details I'm glad to share it. image image

rwols commented 7 months ago

Most likely the logic for terminus doesn’t work anymore since updating to py 3.8

Levitanus commented 7 months ago

I've checked. At the moment everything works, if turn the test for importing terminus off. Needed the new way to check its availability. As hotfix just comment the lines 98-101:

    if not Terminus:
        sublime.error_message(
            'Cannot run executable. You need to install the "Terminus" package and then restart Sublime Text')
        return
WZ-Tong commented 6 months ago

Another workaround (I think will be better):

  1. Package control: Add repository
  2. Add terminus's git repo (https://github.com/randy3k/Terminus/)
  3. Uninstall && Re-install terminus

This will force terminus's version keeps the same with github rather than packagecontrol.io

opedro-c commented 5 months ago

Another workaround (I think will be better):

1. Package control: Add repository

2. Add terminus's git repo (`https://github.com/randy3k/Terminus/`)

3. Uninstall && Re-install terminus

This will force terminus's version keeps the same with github rather than packagecontrol.io

@WZ-Tong Thank you so much! It worked like a charm

predragnikolic commented 1 month ago

Terminus latest release does support 3.8 since August (so the workaround is unnecessary) -> https://github.com/randy3k/Terminus/issues/421#issuecomment-2408627440

So the issue can be closed now