-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Remove gcloud vendor and just use apitools
code
#811
Conversation
and since the other PR is closed -- apitools release is done, so no more worry there. |
f31ae34
to
79332e1
Compare
Thanks for the release Craig! @tseaver PTAL, dont-merge label removed. |
Changes Unknown when pulling 79332e1 on dhermes:remove-gcloud-vendor into * on GoogleCloudPlatform:master*. |
LGTM |
Remove gcloud vendor and just use `apitools` code
@tseaver I'm gonna cut a release today or tomorrow Stay in the |
I'd go with 0.5.0: we're including a new API, for instance. |
See #808 for previous discussion.
Fixes #755.
Labeled dont-merge for now because it depends on an un-released fix in
apitools
./cc @craigcitro