-
Notifications
You must be signed in to change notification settings - Fork 616
Generate gapic client for language v1beta2 #2183
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
Conversation
@landrito I think the coverage change is because you are out of date on your local master. Can you merge master into your PR and push again? |
Done! |
Is there documentation on this versioning scheme, I couldn't find any. I would have thought v1beta2 preceded v1, but we already have v1. Just want to confirm that we didn't make a mistake somewhere, and our v1 was actually a beta as well. Also, will we drop the current v1 when we switch our code to use v1beta2, or do we leave the versions around for users to access directly (not using the handwritten layer)? |
There is mass confusion on this question. Yes, |
Gotcha, thanks! Just some questions about how to handle the version update, because this PR changes
|
The answers to that are: No, and no. I ended up doing this in #2188 (d'oh, sorry @landrito), and that PR supercedes this one. |
No description provided.