Uh oh!
There was an error while loading. Please reload this page.
Remove explicit annotaion for Mypy to infer type - #32670
Conversation
uranusjr
commented
Jul 18, 2023
Hah, static check is failing. Not unexpected. |
benbuckman
commented
Sep 19, 2023
@uranusjr what's the status of this, can this be fixed/merged? Thanks! |
This allows task() to have the correct return type instead of being Any and make everything untyped.
b1a3597 to
e45623dCompareeladkal
commented
Oct 27, 2023
Needs another rebase after #35222 |
eladkal
commented
Nov 8, 2023
@uranusjr needs another rebase |
zach-nicoll-wcq
commented
Dec 5, 2023
@uranusjr would love to see this merged! |
uranusjr
commented
Dec 5, 2023
Note that Mypy is not happy about this change. If you would like this to go in, feel free to build on this change and make Mypy pass. |
itstalmeez
commented
Dec 6, 2023
Addresses issue #32384 by updating the return type of task() to the correct type, removing the 'Any' type and ensuring everything is properly typed. The initial inclusion of 'Any' in the return type is unclear, and though I'm uncertain of its original purpose, locally testing with Mypy shows compatibility with this change. Let's monitor the CI results to confirm its effectiveness across different environments. It's worth noting that Mypy has improved significantly over the past year, and this adjustment aligns with the ongoing efforts to enhance type safety. Your feedback and observations are welcome as we evaluate the impact of this modification. Thank you for bringing attention to this issue, and I look forward to collaborating on its resolution. |
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 5 days if no further activity occurs. Thank you for your contributions. |
This allows
task()to have the correct return type instead of being Any and make everything untyped.I’m not sure why
Anywas added in the first place, but Mypy seems to be fine with this change locally on my machine, so let’s see how it goes in CI. It did improve a lot since a year ago.Fix#32384.