bugraoz93 commented on PR #58594: URL: https://github.com/apache/airflow/pull/58594#issuecomment-3567931704
> Thank you for pointing this out. The changes to update the notation have now been applied in the task-sdk areas as well, including the relevant sensor and S3 code. Let me know if anything else needs to be addressed to fully resolve the issue. Thanks for the update! Unfortunately moving calls not enough to make things green. This need a bit more attention. Please see this one https://github.com/apache/airflow/actions/runs/19610654969/job/56157134273?pr=58594 You can test these locally as well to iterate faster and save resources. Wanted to run them to point out main pain points. ```bash uv tool install -e ./dev/breeze breeze testing task-sdk-tests --python 3.10 ``` Another side effect that arise at the time we tried quick patch is this as well https://github.com/apache/airflow/actions/runs/19610654969/job/56157134251?pr=58594 Please check mypy and other faluires too. They are impacted too such as dateutil in time machine also had some updates. Impacted tests https://github.com/apache/airflow/actions/runs/19610654969/job/56157703353?pr=58594 https://github.com/apache/airflow/actions/runs/19610654969/job/56157134164?pr=58594 Mypy https://github.com/apache/airflow/actions/runs/19610654969/job/56157134164?pr=58594 You can use prek to run mypy hooks -- 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]
