Re: [PR] Add use_glob to GCSObjectExistenceSensor [airflow]

2023-12-17 Thread via GitHub
boring-cyborg[bot] commented on PR #34137: URL: https://github.com/apache/airflow/pull/34137#issuecomment-1859306225 Awesome work, congrats on your first merged pull request! You are invited to check our [Issue Tracker](https://github.com/apache/airflow/issues) for additional contributions.

Re: [PR] Add use_glob to GCSObjectExistenceSensor [airflow]

2023-12-17 Thread via GitHub
potiuk merged PR #34137: URL: https://github.com/apache/airflow/pull/34137 -- 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

Re: [PR] Add use_glob to GCSObjectExistenceSensor [airflow]

2023-12-17 Thread via GitHub
A-Costa commented on PR #34137: URL: https://github.com/apache/airflow/pull/34137#issuecomment-1859290045 Yes, the `match_glob` parameter is first exposed in the `9.0.0` release: https://github.com/talkiq/gcloud-aio/releases/tag/storage-9.0.0 I'll add the minimum version -- This

Re: [PR] Add use_glob to GCSObjectExistenceSensor [airflow]

2023-12-17 Thread via GitHub
potiuk commented on PR #34137: URL: https://github.com/apache/airflow/pull/34137#issuecomment-1859265547 Is there a minimum version of gcloud-aio needed for that ? if so we should set >= MIN -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] Add use_glob to GCSObjectExistenceSensor [airflow]

2023-12-17 Thread via GitHub
A-Costa commented on PR #34137: URL: https://github.com/apache/airflow/pull/34137#issuecomment-1859217555 I've added the missing test for GCSBlobTrigger and removed WIP from the title, as this is ready to be reviewed. > You'll need also to bump `gcloud-aio` version in `provider.ya