-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
CLN: 29547 replace old string formatting 3 #31945
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 3 #31945
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.
@simonjayhawkins @WillAyd do we have a poilicy on this? e.g. i don't think its a big deal to have extra f'' on lines that don't need them.
yeah, no big deal since we don't have code check to enforce. my preference would be to not include but would not request changes on a PR that does. however, would accept clean-up PRs that remove unused. |
Ok will clean it since there is no gain and this is a cleanup anyways |
yeah, that's probably the objection here, the intentional adding of the unused string literal prefixes. |
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.
@3vts lgtm pending green. one comment. not blocker.
Thanks @3vts |
I splitted PR #31844 in batches, this is the third
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