-
-
Notifications
You must be signed in to change notification settings - Fork 17.6k
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: Update mypy and pyright #56493
TYP: Update mypy and pyright #56493
Conversation
# note: assumes python env is setup and activated | ||
name: pyright reportGeneralTypeIssues | ||
entry: pyright --skipunannotated -p pyright_reportGeneralTypeIssues.json --level warning |
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.
moved to pyright_reportGeneralTypeIssues.json.
max_value=pd.Timestamp(1900, 1, 1).to_pydatetime(), | ||
min_value=pd.Timestamp( | ||
1900, 1, 1 | ||
).to_pydatetime(), # pyright: ignore[reportGeneralTypeIssues] |
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.
Could theoretically be NaTType
@@ -39,6 +41,7 @@ | |||
"pandas/core/arrays/string_arrow.py", | |||
"pandas/core/arrays/timedeltas.py", | |||
"pandas/core/computation/align.py", | |||
"pandas/core/computation/ops.py", |
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.
This could be removed in 1.1.400
a29eb07
to
758b6b4
Compare
758b6b4
to
dadcf1f
Compare
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 looks like its mostly cleanups. Just a merge conflict
Thanks @twoertwein |
* changes for mypy * changes for pyright * 1.1.400 would require too many difficult changes
The first commit contains changes for only the mypy update.
The second commit contains only changes for pyright 1.1.399 (this commit has three remaining errors caused by a bug fixed in 1.1.400 ).
The third commit tries to get to 1.1.400. One major change in 1.1.400 is that pyright now also infers missing return annotations for partially annotated functions. Many of these functions would require complex overloads or ignore codes. I added a few overloads but for now, I would prefer to update to 1.1.399 (and ignore the error fixed in 1.1.400).