[GitHub] [airflow] NaveenGokavarapu19 commented on a diff in pull request #24724: Added Support in ts_nodash

[GitBox <gi...@apache.org>]

NaveenGokavarapu19 commented on code in PR #24724:
URL: https://github.com/apache/airflow/pull/24724#discussion_r910109685


##########
airflow/templates.py:
##########
@@ -56,9 +56,11 @@ def ts_filter(value):
     return value.isoformat()
 
 
-def ts_nodash_filter(value):
-    return value.strftime('%Y%m%dT%H%M%S')
-
+def ts_nodash_filter(*value):
+    if value:
+      return value.strftime('%Y%m%dT%H%M%S')
+    else:
+      return None

Review Comment:
   if thought if we take the value as variable args it would work since if an empty list is passed or with values . And what is the meaning of falsy. Any docs or any blog to get more information about this.



-- 
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