vincbeck commented on code in PR #39151:
URL: https://github.com/apache/airflow/pull/39151#discussion_r1576471062


##########
airflow/utils/helpers.py:
##########
@@ -294,6 +294,21 @@ def render_template_as_native(template: jinja2.Template, 
context: Context) -> An
     return render_template(template, cast(MutableMapping[str, Any], context), 
native=True)
 
 
+def at_least_one(*args) -> bool:

Review Comment:
   This function makes the test fail because this code is part of core Airflow 
but is used in the Amazon provider package. Core Airflow and Amazon provider 
package are releases separately, therefore you add a dependency between the two 
packages. There is a solution for that, but as mentioned in my previous 
comment, this function is useless to me.



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

Reply via email to