HaloKo4 commented on a change in pull request #18964:
URL: https://github.com/apache/airflow/pull/18964#discussion_r730032286
##########
File path: airflow/providers/microsoft/azure/sensors/wasb.py
##########
@@ -57,7 +57,7 @@ def __init__(
self.check_options = check_options
def poke(self, context: dict):
- self.log.info('Poking for blob: %s\nin wasb://%s', self.blob_name,
self.container_name)
+ self.log.info('Poking for blob: %s\n in wasb://%s', self.blob_name,
self.container_name)
Review comment:
But this doesn't fix the problem.
The next person who will do `/nsomething` in a log string will be hit by
this pre-commit and might change it to space just like was done on this PR. If
two lines is what you want then there is need to be another pre commit that
prevent using /n in logging. isn't it?
--
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]
