ocket8888 commented on a change in pull request #5209:
URL: https://github.com/apache/trafficcontrol/pull/5209#discussion_r574872972
##########
File path: traffic_control/clients/python/pylint.rc
##########
@@ -259,7 +259,7 @@
redefining-builtins-modules=six.moves,past.builtins,future.builtins,builtins,io
# Format style used to check logging format string. `old` means using %
# formatting, while `new` is for `{}` formatting.
-logging-format-style=new
+logging-format-style=old
Review comment:
The new logging format isn't actually supported, Python the project has
said they're just going to continue using the old format to avoid breaking
people. I don't remember specifics, but I do remember being extremely annoyed
by an inability to use the "new" format.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
