-
Notifications
You must be signed in to change notification settings - Fork 125
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Language server compilation error and no syntax highlighting in vscode when code is valid on Windows #1251
Comments
Thank you for your feedback. There are indeed bugs in version 0.8.5 of the language server, which have been fixed. We will release version 0.8.6 later. cc @He1pa |
Closed by #1247 and released kcl-language-server v0.8.6 |
@Peefy is this supposed to be fixed in kcl-language-server v0.8.6? I just bumped kcl to v0.8.7 and kcl-language-server to v0.8.6 and still seeing the same issue with my test repo (pull latest as I removed a file from the "language-error" branch. |
cc @He1pa Can you help investigate it on the Windows platform. The code repo is here. https://github.com/markphillips100/kcllsp-test I was unable to successfully reproduce the issue. |
Thank you! @markphillips100 We will fix it soon. ❤ |
Closed by #1283, will release kcl-language-server v0.8.7 soon. |
Hi @Peefy kcl-language-server v0.8.7 appears to fix the current test repo, however my real code is still throwing errors in the language server - usual "compile failed code: -32603". Something else causing a problem now I suspect. I'll see if I can narrow it down and update the test repo. |
Thank you @markphillips100 Could you please give more language server logs? |
Yep. I'll copy code for one of my compositions into a new test repo, the smallest one that breaks, and reference here when available with logs attached. |
@Peefy I've added branch error2 and included one of my real compositions. Note, there seems to be a separate issue in regards to some schema complaints around What I've observed is that if I close any open files, close VSCode, reopen it and wait a few seconds then select main.k, syntax highlighting appears, and if I select each file in the project it'll show correct highlighting. If I do the same but open the resources/notification.k file, or any of the files in resources actually, then the language errors start. Here's log output:
|
cc @He1pa Can you help investigate this issue? |
Closed by #1301 and release kcl-language-server v0.9.0-alpha.1 |
@Peefy I'll give it go tomorrow and give you feedback. |
I also fixed the issue you mentioned which falsely report error about schema config |
Yes thank you. I did notice that was working now too. Great work |
1. Minimal reproduce step (Required)
I've added a minimal repro sample here that triggers error popups in VSCode along with no syntax highlighting, even though the code is valid (I think).
It "feels" like the server has a problem with the
filter
statement but this could be just a false observation on my part.Whenever the issue arises and I make some change to rectify, I always have to restart VSCode to test if the server is okay again.
2. What did you expect to see? (Required)
Syntax highlighting should work even if there is a compilation error and indicate what the error is to the user.
If I comment the code in
resources/contributorAssignees.k
and restart VSCode,main.k
shows the following expected highlighting and no errors from language server.3. What did you see instead (Required)
I see errors under the kcl language server output window:
main.k
shows no coloured highlighting.4. What is your KCL components version? (Required)
kcl --version: 0.8.6
kcl-language-server: 0.8.5
VSCode extension "KCL" by kcl_extension_publisher v0.1.6
VSCode v1.88.1
Windows 11
The text was updated successfully, but these errors were encountered: