-
Notifications
You must be signed in to change notification settings - Fork 166
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
[DOC] Add MWE to 3 more functions #1006
Conversation
🚀 Deployed on https://deploy-preview-1006--pyjanitor.netlify.app |
@thatlittleboy looks like we have one error that relates to syntax. Here is a direct link to the logs: https://github.com/pyjanitor-devs/pyjanitor/runs/5069210330?check_suite_focus=true#step:4:1617 |
Codecov Report
@@ Coverage Diff @@
## dev #1006 +/- ##
=======================================
Coverage 96.32% 96.32%
=======================================
Files 77 77
Lines 2858 2858
=======================================
Hits 2753 2753
Misses 105 105 |
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 approve! We're inching closer to finishing!
PR Description
Please describe the changes proposed in the pull request:
functions/row_to_names
,functions/shuffle
,functions/round_to_fraction
.fill_empty.py
(leftover from the previous migration)Resolves #1004 ; Related to #972 .
Aside:
Bugfix for issue 1005 (also relating to the
round_to_fraction.py
file) will be in another PR, I wanted to keep this PR more focused on the MWE additions.I'll update the CHANGELOG for 1004 together with the fix for 1005 in that PR.