-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
[MRG] f-string updates for issue #29547 #31556
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
…ape/pivot, reshape/reshape
Hello @abbiepopa! 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-02-03 04:07:53 UTC |
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 for the PR! Mostly minor edits
Co-Authored-By: William Ayd <[email protected]>
Co-Authored-By: William Ayd <[email protected]>
Co-Authored-By: William Ayd <[email protected]>
Co-Authored-By: William Ayd <[email protected]>
Co-Authored-By: William Ayd <[email protected]>
Co-Authored-By: William Ayd <[email protected]>
Thank you @WillAyd , I covered the changes you requested. I'm having some trouble figuring out why I'm failing the current tests. LMK if it's obvious to you, otherwise I will continue investigating. |
Thanks, Abbie!
If you click on 'details' next to, say, "CI / Checks", you'll see an 'X' near 'linting'. If you expand it, you'll see
So for a start, you need to run
The contributing guide has some more info about this. |
Thank you, @MarcoGorelli ! Looks like the latest failure is a timeout. |
Yay! Thanks @WillAyd and @MarcoGorelli . LMK if you need additional changes etc. |
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.
comment, pls ping on green.
Thanks @jreback and @MomIsBestFriend ! I updated based on your reviews. |
Great thanks @abbiepopa |
Addresses, in part, #29547
…ape/pivot, reshape/reshape
black pandas
git diff upstream/master -u -- "*.py" | flake8 --diff