RE: 
https://github.com/apache/airflow/blob/main/providers/common/compat/src/airflow/providers/common/compat/version_compat.py


Curious if there is a reason we went with the constants like this?  There's 
likely a reason, but if not, we are now adding a constant for 3.3 and that will 
continue to grow.  I'm wondering if there is a reason not to replace those 
constants with a helper like

def airflow_version_at_least(major: int, minor:int,  patch:int = 0) -> bool:
     return get_base_airflow_version_tuple() >= (major, minor, patch)


Then in code instead of importing the constant it would be `if 
airflow_version_at_least(3, 3):` which seems just as clear and easy.

- ferruzzi

Reply via email to