-
Notifications
You must be signed in to change notification settings - Fork 89
Windows support #113
Comments
+1 windows support needed |
👍 |
Doesn't work on Windows, though - their path-handling is all over the place (sourcegraph/go-langserver#113). Will try again when things have stabilised.
I see several Windows PRs merged. Is this issue still relevant? |
In theory it is working, we are running a windows based CI and the tests pass. Can you try the language server out on windows and tell me if it works? I don't have access to a windows machine. |
@keegancsmith This is great news. I gave this a try today from VS Code and found the below:
|
Repeating my comment here, I tested this using VSCode on Windows and everything worked except for Find References:
|
Thanks @doxxx! I can confirm the same. |
FYI, I have submitted a PR fixing the Find References bug on Windows in I tried searching the |
We have file path issues all over the place. We also have some in-progress work to sort this all out by @alexsaveliev. #86 #79
The text was updated successfully, but these errors were encountered: