[PR] Change type definition for `provider_info_cache` decorator [airflow]

2024-05-22 Thread via GitHub
Taragolis opened a new pull request, #39750: URL: https://github.com/apache/airflow/pull/39750 This part decoupled from the https://github.com/apache/airflow/pull/39430, see for details: https://github.com/apache/airflow/pull/39430#discussion_r1590962541 ---

Re: [PR] Change type definition for `provider_info_cache` decorator [airflow]

2024-05-22 Thread via GitHub
potiuk commented on PR #39750: URL: https://github.com/apache/airflow/pull/39750#issuecomment-2124938439 Added `full test needed` just in case. Indeed it should work quite nice with TYPE_CHECKING now, but let's make sure all tests pass (and the STATICA_HACK was there since I remember :). N

Re: [PR] Change type definition for `provider_info_cache` decorator [airflow]

2024-05-22 Thread via GitHub
Taragolis commented on PR #39750: URL: https://github.com/apache/airflow/pull/39750#issuecomment-2124968828 I guess [`STATICA_HACK`](https://stackoverflow.com/questions/20065164/type-hinting-class-not-yet-imported) grabbed originally from the celery repo / SO because there is no TYPE_CHECKI

Re: [PR] Change type definition for `provider_info_cache` decorator [airflow]

2024-05-22 Thread via GitHub
Taragolis commented on PR #39750: URL: https://github.com/apache/airflow/pull/39750#issuecomment-2124977376 In the other places it will be removed by https://github.com/apache/airflow/pull/39430 -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] Change type definition for `provider_info_cache` decorator [airflow]

2024-05-22 Thread via GitHub
Taragolis commented on PR #39750: URL: https://github.com/apache/airflow/pull/39750#issuecomment-2125039469 Only `Tests / Special tests / Pydantic removed test / All:Pydantic-Removed-Postgres:12` which is not related to this changes ```console ERROR collecting tests/providers/

Re: [PR] Change type definition for `provider_info_cache` decorator [airflow]

2024-05-22 Thread via GitHub
Taragolis commented on PR #39750: URL: https://github.com/apache/airflow/pull/39750#issuecomment-2125063979 - https://github.com/apache/airflow/pull/39763 -- 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 t

Re: [PR] Change type definition for `provider_info_cache` decorator [airflow]

2024-05-22 Thread via GitHub
potiuk commented on PR #39750: URL: https://github.com/apache/airflow/pull/39750#issuecomment-2125154689 > I guess [STATICA_HACK](https://stackoverflow.com/questions/20065164/type-hinting-class-not-yet-imported) grabbed originally from the celery repo / SO because there is no TYPE_CHECKING

Re: [PR] Change type definition for `provider_info_cache` decorator [airflow]

2024-05-26 Thread via GitHub
potiuk merged PR #39750: URL: https://github.com/apache/airflow/pull/39750 -- 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.a