potiuk commented on PR #53149:
URL: https://github.com/apache/airflow/pull/53149#issuecomment-3106320375

   > A general miss imo. For public docs etc we should use the public API path: 
`airflow.sdk` and internally use the canonical imports: `from 
airflow.sdk._shared.timezones import timezone`
   
   I think that should be ruff rule. Everything "in" task-sdk should use 
"shared" or "relative" if inside. For rverything "using" "task.sdk" it should 
use the facade. The question is can we somehow forbid (or discourage) importing 
from `_shared" from outside ? 


-- 
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: commits-unsubscr...@airflow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to