potiuk commented on PR #31379: URL: https://github.com/apache/airflow/pull/31379#issuecomment-1552941703
> @potiuk I think try to import `airflow.__version__` and fall back to `airflow.version.version` should work fine -- we can catch the circular error import just like any other error can't we? And what shoudl we do then :) ? It will not serve it purpose, becaue it will also fail with the same recursion error on 2.3 or 2.2 likely. so we will not be able to detect what we wanted to detect (specifically if someone tries to import new sftp or imap provider on Airflow < 2.4 (that was the reason of this change to detect this case) -- 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]
