amoghrajesh commented on code in PR #53791:
URL: https://github.com/apache/airflow/pull/53791#discussion_r2242464895
##########
providers/standard/src/airflow/providers/standard/sensors/time.py:
##########
@@ -42,7 +42,10 @@ class StartTriggerArgs: # type: ignore[no-redef]
timeout: datetime.timedelta | None = None
-from airflow.utils import timezone
+try:
+ from airflow.sdk import timezone
Review Comment:
It's not a protected member per say but part of the shared library / common
code that is shared across distros of Airflow. It was added as part of #53149.
That said, I thought this was sdk file that was changed but its a provider
which is intended to be a consumer of the sdk, therefore the import is correct.
The airflow.sdk.timezone module is a re-export layer of `
airflow.sdk._shared.timezones`, so it is alright. Correcting myself, all good
here.
--
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]
