-
Notifications
You must be signed in to change notification settings - Fork 436
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 cloud-core minimum dependency #786
Conversation
To fix the issue: googleapis#782
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed, please reply here (e.g.
|
I signed it! |
CLAs look good, thanks! |
Thanks for the contribution, @jackchuka. After discussing with @dwsupplee, we feel it's best to bump the required version for all components to the latest I'm quite interested in hearing your thoughts on this idea, as a user of the library who has encountered an issue which our proposal will effect. Would our proposal negatively impact you in any way? |
Hi @jdpedrie, thanks for hearing me out on this. |
@jackchuka please take a look at #790! |
@jackchuka I'm planning on releasing it in v0.47.0, which should be out later today if all goes as planned. |
Source-Link: googleapis/googleapis@f64a3c8 Source-Link: googleapis/googleapis-gen@808541b Copy-Tag: eyJwIjoiQ29tcHV0ZS8uT3dsQm90LnlhbWwiLCJoIjoiODA4NTQxYjdlNzY5N2UyOTg0YTBlYTBlYmExYWRiNTU2MGRhZGU5NyJ9
Source-Link: googleapis/googleapis@f64a3c8 Source-Link: googleapis/googleapis-gen@808541b Copy-Tag: eyJwIjoiQ29tcHV0ZS8uT3dsQm90LnlhbWwiLCJoIjoiODA4NTQxYjdlNzY5N2UyOTg0YTBlYTBlYmExYWRiNTU2MGRhZGU5NyJ9
To fix the issue: #782