search

nim-lang / langserver

The Nim language server implementation (based on nimsuggest)
200 stars 29 forks source link

Update Nimble file checks to work with Atlas and Nimbus build system #154

Open elcritch opened 8 months ago

elcritch commented 8 months ago

Some projects don't / can't use Nimble. It'd be great to have support for projects using Atlas (or the older Nimbus build system). My previous tests with nimlangserver resulted in a bunch of nimble errors occuring.

Idealling this could be done by modifying to check for a nim.cfg which specifies a --noNimblePath switch. I'm not sure if Nim's dump config will specifies that or not.

Alternatively, a nimlangserver config to manually disable using Nimble would work as well.

elcritch commented 8 months ago

I also wonder if it's possible to replace the getProjectFileAutoGuess with running nim to dump the path configs?

Would running nim dump --verbosity:0 --dump.format:json dump.json | jq give all the path info necessary? It seems like it's just trying to find paths, but I haven't dug into getProjectFileAutoGuess to see exactly what it's getting.

dreamflow commented 8 months ago

nimlangserver binaries don't need to be compiled anymore for common platforms . after each commit , automatically compiled fresh new binaries are available at https://github.com/nim-lang/langserver/releases

elcritch commented 8 months ago

nimlangserver binaries don't need to be compiled anymore for common platforms . after each commit , automatically compiled fresh new binaries are available at https://github.com/nim-lang/langserver/releases

Did you mean to post this elsewhere @dreamflow?

dreamflow commented 8 months ago

Did you mean to post this elsewhere @dreamflow?

no , but seems like i misunderstood your first question( nimble , atlas , older build system , ... ) .