Skip to content

[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

Merged
merged 21 commits into from
Feb 4, 2020
Merged

Conversation

abbiepopa
Copy link
Contributor

@abbiepopa abbiepopa commented Feb 2, 2020

Addresses, in part, #29547

…ape/pivot, reshape/reshape

@pep8speaks
Copy link

pep8speaks commented Feb 2, 2020

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

Copy link
Member

@WillAyd WillAyd left a 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

@abbiepopa
Copy link
Contributor Author

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.

@MarcoGorelli
Copy link
Member

Thanks, Abbie!

I'm having some trouble figuring out why I'm failing the current tests

If you click on 'details' next to, say, "CI / Checks", you'll see an 'X' near 'linting'. If you expand it, you'll see

black --version
black, version 19.10b0
Checking black formatting
would reformat /home/runner/work/pandas/pandas/pandas/core/reshape/concat.py
would reformat /home/runner/work/pandas/pandas/pandas/core/reshape/melt.py
would reformat /home/runner/work/pandas/pandas/pandas/core/reshape/pivot.py
would reformat /home/runner/work/pandas/pandas/pandas/core/reshape/merge.py
Oh no! 💥 💔 💥
4 files would be reformatted, 923 files would be left unchanged.

So for a start, you need to run black again any file you've changed. For example,

black pandas/core/reshape/concat.py

The contributing guide has some more info about this.

@abbiepopa
Copy link
Contributor Author

abbiepopa commented Feb 2, 2020

Thank you, @MarcoGorelli ! Looks like the latest failure is a timeout. Is there a way to retrigger the test that timed out? Never mind, I found some instructions in the contributor wiki, retrying now.

@abbiepopa
Copy link
Contributor Author

Yay! Thanks @WillAyd and @MarcoGorelli . LMK if you need additional changes etc.

Copy link
Contributor

@jreback jreback left a 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.

@jreback jreback added the Code Style Code style, linting, code_checks label Feb 2, 2020
@jreback jreback added this to the 1.1 milestone Feb 2, 2020
@abbiepopa
Copy link
Contributor Author

Thanks @jreback and @MomIsBestFriend ! I updated based on your reviews.

@WillAyd WillAyd merged commit 106ed85 into pandas-dev:master Feb 4, 2020
@WillAyd
Copy link
Member

WillAyd commented Feb 4, 2020

Great thanks @abbiepopa

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code Style Code style, linting, code_checks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants