Skip to content

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