saem / vscode-nim

Other
180 stars 23 forks source link

Incorrect configuration schema #105

Open rotu opened 1 year ago

rotu commented 1 year ago

In this project, nim.projectMapping is defined as an object: https://github.com/saem/vscode-nim/blob/a937ed1eca0b81c93b3f6c5dcbbc9c9eb5e2a8de/package.json#L202-L207

But in the language server, it's defined as a sequence/list: https://github.com/nim-lang/langserver/blob/4878cbce6fb0b097f2dcef7cd6d3e5d494bf278a/nimlangserver.nim#L22 https://github.com/nim-lang/langserver/blob/3fcf4c87904ec691ebca035872a4913000fa85df/README.md?plain=1#L40-L44

This mismatch causes either:

This should be corrected and a more descriptive error message should happen when the config is broken.

rotu commented 1 year ago

Possibly responsible for:

https://github.com/saem/vscode-nim/issues/102 https://github.com/saem/vscode-nim/issues/103