[GitHub] [airflow] potiuk commented on pull request #33455: Replace strftime with f-strings where nicer

["potiuk (via GitHub)" <gi...@apache.org>]

potiuk commented on PR #33455:
URL: https://github.com/apache/airflow/pull/33455#issuecomment-1685353375

   > I’m not particular sure about this, strftime is more obvious and (subjectively) easier to read. The f-string format can be beneficial if the formatted date string is going into another f-string (as in some cases here) but not when it’s on its own.
   
   I really like conciseness of the way it's done now. It very nicely focuses the concerns (date + format) in one place within the larger f-string. @uranusjr -> is that good for you as well?


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org