Skip to content

TYP: tighten return type in function any #46638

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

Merged
merged 9 commits into from
May 7, 2022

Conversation

yadav-sachin
Copy link
Contributor

Continuing remaining work after Pull request #44896.
DataFrame.any the return type should be DataFrame or Series, and for Series.any the return type should be Series or bool.

@yadav-sachin
Copy link
Contributor Author

@MarcoGorelli @twoertwein

@MarcoGorelli MarcoGorelli self-requested a review April 4, 2022 20:30
@jreback jreback added the Typing type annotations, mypy/pyright type checking label Apr 10, 2022
@MarcoGorelli MarcoGorelli self-requested a review May 2, 2022 11:58
@yadav-sachin
Copy link
Contributor Author

An entry in the whatsnew is not required for this, right?

Copy link
Member

@twoertwein twoertwein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @yadav-sachin! Except for the small comment, it looks good to me.

Looking forward to @MarcoGorelli's cat GIF :)

Copy link
Member

@twoertwein twoertwein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see comment

Copy link
Member

@twoertwein twoertwein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, thank you @yadav-sachin!

@jreback jreback added this to the 1.5 milestone May 7, 2022
@jreback jreback merged commit 4d9439e into pandas-dev:main May 7, 2022
@yadav-sachin yadav-sachin deleted the any_return_type2 branch May 7, 2022 12:41
@yadav-sachin
Copy link
Contributor Author

Thanks @twoertwein @MarcoGorelli for the reviews.

Waiting for more Cat GIFs. 😺

cat_pouncing_on_fish

@MarcoGorelli
Copy link
Member

😄 the gifs on approval are from https://youtu.be/2rDNJvHznrM

yehoshuadimarsky pushed a commit to yehoshuadimarsky/pandas that referenced this pull request Jul 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Typing type annotations, mypy/pyright type checking
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TYP tighten any return type
4 participants