-
-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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
MAINT: Clean up type: ignore
comments related to third-party packages
#15625
Conversation
`cPickle` was removed back in python 3.0
…i`, rather than inline
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.
I read through the diff and the cleanup looks useful.
My only concern is the comment I added for pyproject.toml
where there may be a bit of duplication.
import cPickle as pickle | ||
except ImportError: | ||
import pickle | ||
import pickle |
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.
ah, leftovers from Python 2.7!
…heir annotations Partially reverts scipy@304db39
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.
LGTM now, thanks @BvB93.
The failing CI job is due to the doc build no longer doing LaTeX - unrelated to this PR and already resolved in main
. So in it goes.
Currently a number of
# type: ignore[import]
comments are scattered throughout the codebase due to third-party packages that lack type annotations. As they provide a reasonable amount of visual clutter, this PR attempts to remove them and deal with the fallout using one of two approaches:types-psutil
).ignore_missing_imports
option for the package in mypy.ini.