-
-
Notifications
You must be signed in to change notification settings - Fork 18.5k
DOC: Updates Index.any docstring from #40362 #41726
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
Conversation
Beluuuuuuga
commented
May 29, 2021
•
edited
Loading
edited
- closes DOC: Wrong examples #40362: add an introduction to all() function as described in "See also"
- tests added / passed
- Ensure all linting tests pass, see here for how to run them
- whatsnew entry
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.
@Beluuuuuuga from the OP
The examples of 'pandas.Index.any' are also included in its own page.
I think my preference would be to delete the any
examples from the all
method docstring.
If we are adding as here, can we shares the examples instead to make them consistent.
Can I just commit the deleted example as well as the one I added this time? |
you can push changes to the same PR as it's the same issue. |
I'll commit this week. |
This pull request is stale because it has been open for thirty days with no activity. Please update or respond to this comment if you're still interested in working on this. |
@Beluuuuuuga is this still active? |
Sorry for the delay. I'll do it tomorrow. |
sure, no hurry |
Thanks for the PR, but it appears to have gone stale. Closing due to inactivity but feel free to push updates to your branch and when you're ready for another review we can reopen. |