-
Notifications
You must be signed in to change notification settings - Fork 457
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
python: Remove unnecessary f-strings #18821
Conversation
+1 for enabling the linter rule |
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.
Yes please enable the linter rule -- if you felt strong enough about this to go through the existing codebase and fix stuff, then we should keep enforcing it going forward.
It was just a quick grep after I noticed it a few times randomly before. |
New commit with the linter rule added. |
We could also enable pylint's f-string-without-interpolation / W1309 https://pylint.readthedocs.io/en/latest/user_guide/messages/warning/f-string-without-interpolation.html
Has been disabled since the start: MaterializeInc#7845
Forgot to switch the escaped {{}} strings which are sent to testdrive, follow-up to MaterializeInc#18821
Forgot to switch the escaped {{}} strings which are sent to testdrive, follow-up to MaterializeInc#18821
Forgot to switch the escaped {{}} strings which are sent to testdrive, follow-up to MaterializeInc#18821
We could also enable something like pylint's f-string-without-interpolation / W1309
https://pylint.readthedocs.io/en/latest/user_guide/messages/warning/f-string-without-interpolation.html
Motivation
Checklist
$T ⇔ Proto$T
mapping (possibly in a backwards-incompatible way) and therefore is tagged with aT-proto
label.