-
Notifications
You must be signed in to change notification settings - Fork 675
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
"import xxxxx" without any warnings in the error list. #7687
Comments
@ttSpace, I can't repro this issue. With the latest Preview 4 build I see an import resolve error. Can you share the output for the 'Pylance' server? |
I can repro on Microsoft Visual Studio Enterprise 2022 (64-bit) - Int Preview Version 17.8.0 Preview 2.0 [33930.147.main]. This is the link to download the latest int preview: https://aka.ms/vs/17/intpreview/vs_enterprise.exe Below are the logs: Info: (15896) Pylance language server 2023.7.40 (pyright b917b7a5) starting |
Wouldn't the fix then be to upgrade to Preview 4? Can you repro on Preview 4? |
I'm trying to install Preview 2. |
Nevermind I can't even run that version. Well at least build it locally because the references are out of date. |
ok I ran into the same errors while building it too. At first I thought it might be related to the version number in main being lower than the release branch, but just checked that the version numbers in both branches are the same, so I am not sure what's causing the issue. I was able to work around it by searching for the ids in the error messages in PTVS, and manually bumping up the version number to something like |
@rchiodo The output of Pylance. LogsOutput for
|
Oh I'm using 17.7. You two are using 17.8. That might be the problem. It could be an issue with something in Visual Studio. Let me try installing 17.8. |
I can repro now. With pylance 2023.7.30 and 17.8 |
Internally this is happening because the 'reportMissingImports' diagnostic severity is set to 'none'. It's like ALL of the errors are turned off. Not sure why yet. |
It's because Pylance thinks the workspace is not local. It looks like VS is not sending a URI for the workspace root anymore and just sending a file path. |
Found the root cause. Change in the DocumentUriConverter by core. Bug has been fixed but is not shipping yet: |
Closing as it doesn't repro anymore on Microsoft Visual Studio Enterprise 2022 (64-bit) - Int Preview Version 17.8.0 Preview 2.0 [34002.499.main] |
Describe the bug
Steps to Reproduce
import xxxxx
.Expected behavior
Displays a green squiggly line under xxxxx and a warning in the error list.
Additional context and screenshots
No green squiggly lines and warnings.
The text was updated successfully, but these errors were encountered: