-
-
Notifications
You must be signed in to change notification settings - Fork 18.7k
DOC: Delete trailing blank lines in docstrings. #23651
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
Hello @douglatornell! Thanks for submitting the 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.
lgtm, thanks @douglatornell
Is this addressing all the cases in the issue? If it is you can check the checkbox in the PR description, so the issue is closed when we merge this. If it is not, you can replace the closes
by a refs
, so it's clear it doesn't.
Codecov Report
@@ Coverage Diff @@
## master #23651 +/- ##
=======================================
Coverage 92.24% 92.24%
=======================================
Files 161 161
Lines 51317 51317
=======================================
Hits 47338 47338
Misses 3979 3979
Continue to review full report at Codecov.
|
@gfyoung, can you take a look here (similar the one you approved before), and merge if you're happy |
Thanks, @douglatornell ! |
re: issue #23632
git diff upstream/master -u -- "*.py" | flake8 --diff