Skip to content
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.

Sign up for GitHub

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

Jump to bottom

(🐞) dmypy inspect doesn't select the right file with multiple files with same name #15781

Closed
KotlinIsland opened this issue Jul 30, 2023 · 0 comments · Fixed by #16357
Closed

(🐞) dmypy inspect doesn't select the right file with multiple files with same name #15781

KotlinIsland opened this issue Jul 30, 2023 · 0 comments · Fixed by #16357
Labels
bug mypy got something wrong

Comments

@KotlinIsland
Copy link
Contributor

test.py

b: str

from demo.test import a

demo/test.py

a: int
👉 dmypy run test.py --export-types
Success: no issues found in 44 source files
👉 dmypy inspect demo/test.py:1:1
"str"
@KotlinIsland KotlinIsland added the bug mypy got something wrong label Jul 30, 2023
@ilevkivskyi ilevkivskyi mentioned this issue Oct 28, 2023
Merged
JukkaL pushed a commit that referenced this issue Oct 30, 2023
@ilevkivskyi
Fix dmypy inspect for namespace packages (#16357)
4e30e89 
Fixes #15781

The fix is to switch to already resolved paths instead of relying on
`crawl_up()`. This should be more robust w.r.t. various special cases. I
also tweak the tests slightly to show full file names, to have a more
consistent output.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug mypy got something wrong
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix dmypy inspect for namespace packages ilevkivskyi/mypy
1 participant
@KotlinIsland