-
-
Notifications
You must be signed in to change notification settings - Fork 18k
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
TYP: investigate/fix ignored mypy errors #37715
Comments
Hello, I'm contributing for the first time to OSS. Can i try this one ? |
@Praveenk8051 some fixes will be easier than others. anyone is welcome to take a look at addressing any of these mypy errors. |
Thank you. I will go through the |
Hi @simonjayhawkins / @Praveenk8051 do you guys need more help ? |
@sidram05 sure. #37715 (comment)
|
@sidram05 I'm not working on this. I'm working on different issue |
@sidram05 Are you working on this ? |
Would be fixed with #38334 |
(Most (i.e. all In turn, it introduced:
I'll try to put together a PR fixing some of the remaining in there. |
#37715 (comment) is now fixed |
updated checkboxes for #37715 (comment), #37715 (comment), #37715 (comment) |
You can simply add a xref when opening your pr |
Is this issue resolved? |
Is this issue open? I would like to work on it |
Hi, I want to work on this issue. Can anybody guide me what to do to fix mypy error ? |
shall we close this? the most obvious ones have been done already, it's generally not clear to people how to get started, and fixing the remaining type ignores requires a fair bit of knowledge |
Agreed. Closing |
In #37556
# type: ignore
were added to silence mypy errors to be fixed 'later'. If an ignore is needed due to a error with the type checker, a comment with a reference to the mypy issue on the github mypy issue tracker should be included.Further investigation or PRs welcome removing these ignores or adding comments with links to mypy issues if applicable. PRs should include 'xref #37715'
PRs could address just one error, a small handful of related errors or a complete module.
grep -Ern "type: ?ignore" pandas/
currently gives 207 matchesThe text was updated successfully, but these errors were encountered: