Re: [PR] Fixes typing in `get_provider_name_from_short_hand` and its uses [airflow]

2023-12-23 Thread via GitHub
github-actions[bot] closed pull request #34507: Fixes typing in `get_provider_name_from_short_hand` and its uses URL: https://github.com/apache/airflow/pull/34507 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

Re: [PR] Fixes typing in `get_provider_name_from_short_hand` and its uses [airflow]

2023-12-17 Thread via GitHub
github-actions[bot] commented on PR #34507: URL: https://github.com/apache/airflow/pull/34507#issuecomment-1859338975 This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 5 days if no further activity occurs. Thank you for you

Re: [PR] Fixes typing in `get_provider_name_from_short_hand` and its uses [airflow]

2023-11-02 Thread via GitHub
potiuk commented on PR #34507: URL: https://github.com/apache/airflow/pull/34507#issuecomment-1790242653 I think there are some errors (see the build failing) also rather than changing method name, I think you should change argument name (short_package_name)? -> the way the method is named

Re: [PR] Fixes typing in `get_provider_name_from_short_hand` and its uses [airflow]

2023-11-01 Thread via GitHub
carterjfulcher commented on PR #34507: URL: https://github.com/apache/airflow/pull/34507#issuecomment-1790009225 > And static checks should be fixed. Fixed @potiuk -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

Re: [PR] Fixes typing in `get_provider_name_from_short_hand` and its uses [airflow]

2023-10-08 Thread via GitHub
potiuk commented on PR #34507: URL: https://github.com/apache/airflow/pull/34507#issuecomment-1752133914 And static checks should be fixed. -- 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 spe

Re: [PR] Fixes typing in `get_provider_name_from_short_hand` and its uses [airflow]

2023-10-04 Thread via GitHub
uranusjr commented on code in PR #34507: URL: https://github.com/apache/airflow/pull/34507#discussion_r1346535068 ## dev/breeze/src/airflow_breeze/utils/publish_docs_helpers.py: ## @@ -112,7 +112,7 @@ def process_package_filters( return available_packages expande

Re: [PR] Fixes typing in `get_provider_name_from_short_hand` and its uses [airflow]

2023-10-04 Thread via GitHub
uranusjr commented on code in PR #34507: URL: https://github.com/apache/airflow/pull/34507#discussion_r1346534457 ## dev/breeze/src/airflow_breeze/utils/publish_docs_helpers.py: ## @@ -112,7 +112,7 @@ def process_package_filters( return available_packages expande

Re: [PR] Fixes typing in `get_provider_name_from_short_hand` and its uses [airflow]

2023-10-04 Thread via GitHub
uranusjr commented on code in PR #34507: URL: https://github.com/apache/airflow/pull/34507#discussion_r1346533744 ## dev/breeze/src/airflow_breeze/utils/publish_docs_helpers.py: ## @@ -112,7 +112,7 @@ def process_package_filters( return available_packages expande

Re: [PR] Fixes typing in `get_provider_name_from_short_hand` and its uses [airflow]

2023-10-04 Thread via GitHub
uranusjr commented on code in PR #34507: URL: https://github.com/apache/airflow/pull/34507#discussion_r1346533744 ## dev/breeze/src/airflow_breeze/utils/publish_docs_helpers.py: ## @@ -112,7 +112,7 @@ def process_package_filters( return available_packages expande

Re: [PR] Fixes typing in `get_provider_name_from_short_hand` and its uses [airflow]

2023-10-04 Thread via GitHub
carterjfulcher commented on PR #34507: URL: https://github.com/apache/airflow/pull/34507#issuecomment-1747635703 > @carterjfulcher can you look into the failing static check? Yes, will do -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] Fixes typing in `get_provider_name_from_short_hand` and its uses [airflow]

2023-10-04 Thread via GitHub
eladkal commented on PR #34507: URL: https://github.com/apache/airflow/pull/34507#issuecomment-1747630776 @carterjfulcher can you look into the failing static check? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the