Prefer f-strings instead of str.format #2164
Conversation
WhyNotHugo
force-pushed
the
f-strings
branch
from
c76c46b to
1e897a4
Compare
| @@ -429,9 +429,9 @@ def get_best_match(self, ireq: InstallRequirement) -> InstallRequirement: | |||
|
|
|||
| # Format the best match | |||
| log.debug( | |||
There was a problem hiding this comment.
Logging is supposed to pass params through dedicated arguments, though. Instead of pre-rendering strings before passing them into logging..
There was a problem hiding this comment.
That's exactly what my change does.
Previously a pre-rendered string was being passed (one using .format()). Now dedicated arguments are passed.
Ruff's pyupgrade re-implementation does handle this. This PR and #2163 are groundwork for using ruff instead of flake8+pyupgrade, but can be merged independently. |
WhyNotHugo
force-pushed
the
f-strings
branch
from
1e897a4 to
a6939e2
Compare
|
Not sure about th CI failures. Rebased onto latest |
|
I think pyupgrade would, indeed, handle a couple of these and is a good idea (original implementation or ruff) regardless for the numerous other fixes it applies. But that's doesn't block this PR. I've hit the GitHub rebase button since a lot of CI has been fixed significantly at this point. |
No user facing changes.