-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
CLN: 29547 replace old string formatting 6 #31980
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: 29547 replace old string formatting 6 #31980
Conversation
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.
Minor nits, otherwise LGTM
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 @3vts lgtm ex @MomIsBestFriend comments.
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 @3vts |
I splitted PR #31844 in batches, this is the sixth
For this PR I ran the command
grep -l -R -e '%s' -e '%d' -e '\.format(' --include=*.{py,pyx} pandas/
and checked all the files that were returned for.format(
and changed the old string format for the correspondingfstrings
to attempt a full clean of, #29547. I may have missed something so is a good idea to double check just in caseblack pandas
git diff upstream/master -u -- "*.py" | flake8 --diff