-
-
Notifications
You must be signed in to change notification settings - Fork 18.3k
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
CLN: replaced Appender with doc #33633
CLN: replaced Appender with doc #33633
Conversation
4394157
to
1b624ea
Compare
Can you fix merge conflicts? |
1b624ea
to
f8566fc
Compare
Can you fix up merge conflict again? If so can merge |
@smartvinnetou can you fix conflicts? |
Will do.
…On Thu, 10 Sep 2020, 20:04 William Ayd, ***@***.***> wrote:
@smartvinnetou <https://github.com/smartvinnetou> can you fix conflicts?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#33633 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AOSDPMQR23NHNI3MBUUW6L3SFEPMHANCNFSM4MLNGF7A>
.
|
one more rebase pls |
f1108b3
to
0e5dc6d
Compare
Hello @smartvinnetou! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found: There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻 Comment last updated at 2020-09-13 19:38:05 UTC |
Rebase done. Still some glitches in the doctests. Will keep fixing. |
51261af
to
a27e3f5
Compare
thanks @smartvinnetou keep em coming |
black pandas
git diff upstream/master -u -- "*.py" | flake8 --diff