-
-
Notifications
You must be signed in to change notification settings - Fork 18.5k
DOC: Removed duplicate doc line #24668
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
Looking at the dev docs I think there are actually a few other issues with this one. Do you mind taking a look? |
Codecov Report
@@ Coverage Diff @@
## master #24668 +/- ##
=======================================
Coverage 92.37% 92.37%
=======================================
Files 166 166
Lines 52315 52315
=======================================
Hits 48327 48327
Misses 3988 3988
Continue to review full report at Codecov.
|
1 similar comment
Codecov Report
@@ Coverage Diff @@
## master #24668 +/- ##
=======================================
Coverage 92.37% 92.37%
=======================================
Files 166 166
Lines 52315 52315
=======================================
Hits 48327 48327
Misses 3988 3988
Continue to review full report at Codecov.
|
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.
See comment above
@palewire do you mind running I'm not sure your fix is correct, as the latest version is different from your screenshot and the link you referred to. Seems like there is a problem with the sections, and those notes are now rendered in the returns section. You may also want to run |
@WillAyd @palewire @datapythonista I've tried to fix the issues. The PR is here #24948. Please let me know if anything is amiss. |
close this in favor of #24948, a more comprehensive fix. |
This line of documentation is already included in the parent shared docs. So the line appears twice. See the screenshot below from the Dataframe.aggregate page. My patch cleans that up.