Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork1.9k
Bump mypy to 1.11.1#12463
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Bump mypy to 1.11.1#12463
Uh oh!
There was an error while loading.Please reload this page.
Conversation
sobolevn commentedJul 31, 2024
There are tons of new errors due to new checks of "untyped code". I will see how easy it would be to fix them. |
sobolevn commentedJul 31, 2024
Well, there are so many errors that I start to think that it might be a regression in mypy:https://github.com/python/typeshed/actions/runs/10187104121/job/28180408403?pr=12463 cc@hauntsaninja |
This comment has been minimized.
This comment has been minimized.
AlexWaygood commentedAug 1, 2024 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
There are also many |
AlexWaygood commentedAug 1, 2024 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
The changes to the The new At first glance I think everything is working as expected here; I doubt there's a mypy regression to report |
srittau commentedAug 1, 2024
We could do#12178 now if that makes transition easier. I don't think it's critical to do it after the mypy update. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
sobolevn commentedAug 3, 2024
@srittau@AlexWaygood any ideas on why |
This comment has been minimized.
This comment has been minimized.
Daverball commentedAug 3, 2024 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Maybe it's because it doesn't have a file ending? So it doesn't know whether it should use the INI or TOML parser, so it doesn't use either and the file is just silently ignored. Is there any reason not to put a Edit: It also makes it easier to discover what checks are enabled/disabled by default in typeshed. |
AlexWaygood commentedAug 3, 2024
@Daverball we've discussed that idea previously in#2852. I think it would honestly just make our config situation more complicated, because different tests are still going to require slightly different options. It'll be hard to tell which options are provided by the global config and which are overridden in the specific test script, and it could mislead contributors into thinking that running mypy on typeshed is simpler than it is. |
sobolevn commentedAug 3, 2024
Nope, explicit extension didn't work: Looks like I will be debugging this manually now :) |
This comment has been minimized.
This comment has been minimized.
sobolevn commentedAug 3, 2024
This is a mypy bug :) |
sobolevn commentedAug 3, 2024
If anyone is interested in reviewing my PR it is here:python/mypy#17629 |
sobolevn commentedAug 4, 2024
I dropped all config changes, because it is not yet supported by stubtest (until next release). |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
sobolevn commentedAug 4, 2024
Ok, the first part is done. Now I will run tests with |
This comment has been minimized.
This comment has been minimized.
sobolevn commentedAug 4, 2024
@srittau@AlexWaygood done! It took me quite some time to finish, but now it is ready! 🎉 |
Diff frommypy_primer, showing the effect of this PR on open source code: bokeh (https://github.com/bokeh/bokeh)- src/bokeh/models/util/structure.py:165: note: ... from here:cwltool (https://github.com/common-workflow-language/cwltool)- note: ... from here,- note: ... from here,- cwltool/utils.py:52: note: ... from here: |
hauntsaninja left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Nice, thank you! Great that this catches a bug in reportlab source too :-)
Let's see how it goes.