-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
DOC: fixed Ex03 errors in docstrings: #56878
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
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.
Thanks @tiffanyxiao, great job. Just one comment, as I think you can simplify the formatting in the see also section.
Also, there seems to be a conflict after merging another PR. Can you merge main into your PR and resolve the conflict please?
pandas/errors/__init__.py
Outdated
|
||
See Also | ||
-------- | ||
:attr:`.pandas.DataFrame.iloc` : Purely integer-location based indexing for \ |
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.
I don't think in the See Also
section we need those :attr:
markers, I think just DataFrame.iloc
should render correctly. Same for the other cases in this PR.
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.
Hi Marc,
Thanks! I pushed these changes up.
Thanks @tiffanyxiao |
* Addressing docstring errors * Resolving merge conflicts * Removing extra formatting in See Also
PR SUMMARY
Checked if validation docstrings passes for:
OUTPUT:
PR CHECKLIST
doc/source/whatsnew/vX.X.X.rst
file if fixing a bug or adding a new feature.