NimPlus icon indicating copy to clipboard operation
NimPlus copied to clipboard

LSP-nimlangserver

Open lewisl opened this issue 1 year ago • 3 comments

Does it also make sense to have LSP-nimlangserver (and sublime lsp) installed, too? To what extent does this overlap with NimPlus?

lewisl avatar Aug 24 '24 05:08 lewisl

NimPlus does not use the Nim LSP for suggestions / checking. You can use it without installing them. On the other-hand, if you prefer the suggestions provided by LSP-nimlangserver, you can disable the suggestions from NimPlus in the settings and use those provided by the LSP.

In my opinion, both suggestions are slow and do not provide quality suggestions. When I get some time (this means in a very long time :'( ), I'll try to write a better (and faster) suggestion engine to improve the DevX that does not rely on the Nim compiler.

vanyle avatar Aug 24 '24 11:08 vanyle

Which option is that? I don’t see only:

"nimplus.savecheck": true,
    // Autocomplete support ? (based on nimsuggest)
    "nimplus.autocomplete": true,
    // Provide description when hovering over symbols (based on nim suggest)
    "nimplus.hoverdescription": true,

    // Arguments to prepend to the nim build commands.
    // This can be used to specify a console, for example using: "wt","--window","0" on windows terminal.
    "nimplus.nim.console": [],
    "nimplus.nimble.console": [],

    // Arguments passed to the nim compiler when building / checking the file
    // The path to the current file is appended at the end.
    // You can add here --d:release and --d:danger when needed
    "nimplus.nim.arguments": ["--threads:on","-d:ssl","--stdout:on"],

    "nimplus.nim.save_before_build": true,
    "nimplus.nimble.save_before_build": true,

    // Set to true if you have terminus install and want to enjoy the integration with it.
    // The output is not displayed in the output panel anymore in this case.
    // Terminus allows you to interact the the stdin of your program which is nice when developping command line applications
    "nimplus.use_terminus": false

My overrides are:

{
    "nimplus.nim.arguments": ["--mm:orc", "-d:release", "-d:lto","--stdout:on"],
    "nimplus.use_terminus": true
}

I don’t care about autocomplete, but checking often picks up things that the compiler would flag…

From: Antoine Delègue @.> Reply-To: vanyle/NimPlus @.> Date: Saturday, August 24, 2024 at 4:08 AM To: vanyle/NimPlus @.> Cc: Lewis Levin @.>, Author @.***> Subject: Re: [vanyle/NimPlus] LSP-nimlangserver (Issue #17)

NimPlus does not use the Nim LSP for suggestions / checking. You can use it without installing them. On the other-hand, if you prefer the suggestions provided by LSP-nimlangserver, you can disable the suggestions from NimPlus in the settings and use those provided by the LSP.

In my opinion, both suggestions are slow and do not provide quality suggestions. When I get some time, I'll try to write a better (and faster) suggestion engine to improve the DevX that does not rely on the Nim compiler.

— Reply to this email directly, view it on GitHubhttps://github.com/vanyle/NimPlus/issues/17#issuecomment-2308356689, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAIYWLMWR5NH5DXOAGSCG2DZTBSSPAVCNFSM6AAAAABNBIKHD6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBYGM2TMNRYHE. You are receiving this because you authored the thread.Message ID: @.***>

lewisl avatar Aug 24 '24 19:08 lewisl

Great approach. I couldn’t care less about autocomplete, but getting compile errors at save time is really useful.

From: Antoine Delègue @.> Reply-To: vanyle/NimPlus @.> Date: Saturday, August 24, 2024 at 4:08 AM To: vanyle/NimPlus @.> Cc: Lewis Levin @.>, Author @.***> Subject: Re: [vanyle/NimPlus] LSP-nimlangserver (Issue #17)

NimPlus does not use the Nim LSP for suggestions / checking. You can use it without installing them. On the other-hand, if you prefer the suggestions provided by LSP-nimlangserver, you can disable the suggestions from NimPlus in the settings and use those provided by the LSP.

In my opinion, both suggestions are slow and do not provide quality suggestions. When I get some time, I'll try to write a better (and faster) suggestion engine to improve the DevX that does not rely on the Nim compiler.

— Reply to this email directly, view it on GitHubhttps://github.com/vanyle/NimPlus/issues/17#issuecomment-2308356689, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAIYWLMWR5NH5DXOAGSCG2DZTBSSPAVCNFSM6AAAAABNBIKHD6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBYGM2TMNRYHE. You are receiving this because you authored the thread.Message ID: @.***>

lewisl avatar Aug 27 '24 16:08 lewisl