sebbASF commented on issue #339: URL: https://github.com/apache/tooling-trusted-releases/issues/339#issuecomment-3569761747
As I wrote: the policy specifically says: > printf style is usually expected, such as in `log.info` calls yet it insists that f-strings be used instead. To my mind, this is 'un-usual', i.e. not 'usual'. AIUI, the logging interface is specifically designed to avoid unnecessary construction of strings, so the fact that the policy goes against this is a decision that needs to be explained. A policy without a rationale cannot be evaluated and is harder to maintain if circumstances change. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
