potiuk commented on PR #37585: URL: https://github.com/apache/airflow/pull/37585#issuecomment-1956401751
> While you're at it, perhaps also modify the PENDULUM3 definitions in > airflow/utils/timezone.py > tests/serialization/serializers/test_serializers.py Ah. Nice. I missed those, yes, good idea to fix them there - we had recently a long discussion about using `__version__` with @Taragolis https://github.com/apache/airflow/pull/37320#discussion_r1486723855 and we came to the conclusion is that we should indeed switch to using importlib/metadata instead as the only "standard" way, I even have a draft PR to change the approach we use in our providers https://github.com/apache/airflow/pull/37335 as result of that (waited for the release of providers that just happened so I might revive it). -- 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]
