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

fix: do not throw DeprecationWarnings for legacy retry behavior #1551

Merged
merged 1 commit into from
Jan 31, 2024

Conversation

alexander-fenster
Copy link
Contributor

If a client library uses legacy retry behavior, the deprecation warning is useless because the user who receives it cannot do anything about it. We don't need to throw it at this point.

@product-auto-label product-auto-label bot added the size: m Pull request size is medium. label Jan 31, 2024
@alexander-fenster alexander-fenster merged commit 2f39306 into main Jan 31, 2024
21 checks passed
@alexander-fenster alexander-fenster deleted the no-dep-warn branch January 31, 2024 18:44
@release-please release-please bot mentioned this pull request Jan 31, 2024
gcf-merge-on-green bot pushed a commit that referenced this pull request Jan 31, 2024
🤖 I have created a release *beep* *boop*
---


<details><summary>4.3.0</summary>

## [4.3.0](https://togithub.com/googleapis/gax-nodejs/compare/v4.2.1...v4.3.0) (2024-01-31)


### Features

* implement support for Trusted Partner Cloud ([#1552](https://togithub.com/googleapis/gax-nodejs/issues/1552)) ([d51218c](https://togithub.com/googleapis/gax-nodejs/commit/d51218c23d4afe437285a55a44a39f64bdb3736d))


### Bug Fixes

* do not throw DeprecationWarnings for legacy retry behavior ([#1551](https://togithub.com/googleapis/gax-nodejs/issues/1551)) ([2f39306](https://togithub.com/googleapis/gax-nodejs/commit/2f39306a441ce08374fe069ab1e1892871715a77))
</details>

---
This PR was generated with [Release Please](https://togithub.com/googleapis/release-please). See [documentation](https://togithub.com/googleapis/release-please#release-please).
@release-please release-please bot mentioned this pull request Feb 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size: m Pull request size is medium.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants