Skip to content

Fixing RT02 pandas.Index.dropna and PR08 pandas.Index.fillna #32359

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 3 commits into from
Mar 6, 2020

Conversation

zaki-indra
Copy link
Contributor

Copy link
Member

@WillAyd WillAyd left a comment

Choose a reason for hiding this comment

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

Looks good though failing CI - I think comments should help address

@@ -2159,7 +2159,8 @@ def dropna(self, how="any"):

Returns
-------
valid : Index
valid
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
valid
Index

Can just list the type of object being returned here

@@ -2159,7 +2159,8 @@ def dropna(self, how="any"):

Returns
-------
valid : Index
valid
Index
Copy link
Member

Choose a reason for hiding this comment

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

Delete this line

@WillAyd WillAyd added this to the 1.1 milestone Mar 6, 2020
@WillAyd WillAyd merged commit 96a40ed into pandas-dev:master Mar 6, 2020
@WillAyd
Copy link
Member

WillAyd commented Mar 6, 2020

Thanks @zaki-indra

SeeminSyed pushed a commit to CSCD01-team01/pandas that referenced this pull request Mar 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants