potiuk commented on PR #53722: URL: https://github.com/apache/airflow/pull/53722#issuecomment-3905323464
> @xBis7 your assumption is correct, providers need to get it from task sdk but not from airflow.sdk._shared since its an internal path. This PR: https://github.com/apache/airflow/pull/56187/changes#diff-8603867cd4927a3bbbfe40fd1e3250eff900162514c9b70d41fa8ddfaefc5809 did the right thing but some part of it was lost in rebase I guess Yes. airflow.sdk should expose everything it needs via non `_*` module - I keep on rebasing this one that should make all the checks automatically - but there are still a number of violations that prevent it from being merged (and it's likely some rules will need to be adjusted/refreshed) https://github.com/apache/airflow/pull/58825 -- 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]
