potiuk commented on code in PR #44607:
URL: https://github.com/apache/airflow/pull/44607#discussion_r1869324960


##########
airflow/__init__.py:
##########
@@ -58,10 +58,20 @@
 # very easily cause import cycles in the conf init/validate code (since 
downstream code from
 # those functions likely import settings).
 # configuration is therefore initted early here, simply by importing it.
+
+from packaging.version import Version
+
 from airflow import configuration, settings
 
+

Review Comment:
   Actually, after thinking a bit and seeing it - I think we can have cale And 
wat it too. Maybe we can generate the constants via `__init__.py` template in 
all providers? 
   
   This way each procider could have automatically  generated own set of 
constant (possibly even named differently to avoid confusion where to importit 
from.
   
   
   I am at family funeral today with my mum and two aunt's but I can propose a 
PR when I am back if that seems appealing 



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