-
Notifications
You must be signed in to change notification settings - Fork 35
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
False positives probably related to caching #392
Comments
@DervishD There is a setting The trace setting captures content that is being sent to the server and we should be able to see if this is due to synchronization issue. |
@karthiknadig thanks A LOT for the information and advice. Right now I'm quite busy for testing this properly, so my first try will be disabling |
@karthiknadig a small update: the problem still happens with My next step will be setting the verbose flag and dump the report, as soon as I can. |
@karthiknadig I added If you need me to do a different thing or sending you a different content, please tell me. Also, even though this is by no means a "secret" project (it is, in fact one of my public repositories), the contents of the file may reveal local paths of my development machine which I may want to obscure, if you don't mind, before adding the trace file here. In short, I may need some instructions 👼🏼 Thanks a lot! |
@karthiknadig as soon as I can, probably tomorrow morning before start coding 😄 Thanks A LOT! Anyway, the problem has not happened again, probably because I'm now working with a simpler version of the code (I'm refactoring it) so for any reason it is not triggering the bug. I'll try the new version ASAP and if it happens again I'll have a report ready. |
I have not been able to reproduce the issue with latest version, so I'm closing the issue for the time being, if that is not overstepping. Thanks a lot for the help and the fix! |
I've reported the bug on the
pylint-dev
repo, at pylint-dev/pylint/issues/9047 but I'm reporting it here too because it seems to be related topylint
bein used in server mode by the extension, as in #132 and #129 .You can see the details about my environment, etc. in the linked
pylint-dev
issue. I'm not repeating everything here to save a bit of noise and to not change any information by mistake.The code I've been using to reproduce the code is quite weird, it is already posted in the
pylint-dev
issue, and more importantly, it is useless for reliably reproducing the bug, as it does not happen consistently. Fast saves in long code, long and complex code, etc. seem to trigger the issue, but I cannot reproduce it consistently.I've tested with the embedded interpreter and one installed in my environment and the results are the same: from time to time I get spurious errors (in my case E0710 and E0712) because I think the server is not getting all the code, and the part where the custom exceptions are declared and derived from
Exception
does not get into the server or is ignored or something like that, causing the error to appear.The errors are fixed as soon as I restart the server by hand using the command palette. They reappear sooner than later, but at least they go away for some time.
I'm reporting it here just in case someone searches for the issue, so there is at least some information available in the relevant repositories.
Anyway, if anyone is aware of caching problems or known bugs that can be worked-around, I'm all ears because this is very uncomfortable, having to restart the server or even Visual Studio Code to have proper linting reports.
Thanks a lot in advance and feel free to ask me if more information is needed.
The text was updated successfully, but these errors were encountered: