Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update retry param typing in PubSubAsyncHook #36198

Merged
merged 1 commit into from
Dec 13, 2023

Conversation

josh-fell
Copy link
Contributor

The google-cloud-pubsub library was recently released with a bug fix to update the retry parameter when using the SubscriberAsyncClient to AsyncRetry from Retry.

Main builds were failing Mypy checks for this reason.

The `google-cloud-pubsub` library was [recently released](https://pypi.org/project/google-cloud-pubsub/2.19.0/) with a bug fix to update the `retry` parameter when using the SubscriberAsyncClient to AsyncRetry from Retry.

Main builds were failing Mypy checks for this reason.
@boring-cyborg boring-cyborg bot added area:providers provider:google Google (including GCP) related issues labels Dec 13, 2023
@potiuk
Copy link
Member

potiuk commented Dec 13, 2023

We seen to have another - stackdriver problem (test expects older version of response). I will fix it separately and merge that one first.

@potiuk potiuk merged commit 8fbacb8 into apache:main Dec 13, 2023
72 of 76 checks passed
@josh-fell josh-fell deleted the pubsub-async-retry-typing branch December 13, 2023 14:45
potiuk pushed a commit that referenced this pull request Dec 15, 2023
The `google-cloud-pubsub` library was [recently released](https://pypi.org/project/google-cloud-pubsub/2.19.0/) with a bug fix to update the `retry` parameter when using the SubscriberAsyncClient to AsyncRetry from Retry.

Main builds were failing Mypy checks for this reason.

(cherry picked from commit 8fbacb8)
@ephraimbuddy ephraimbuddy added the changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) label Dec 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:providers changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) provider:google Google (including GCP) related issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants