Skip to content
New issue

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

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

Already on GitHub? # to your account

chore: support string type response.data for gcloud #503

Merged
merged 1 commit into from
May 6, 2020

Conversation

arithmetic1728
Copy link
Contributor

gcloud uses httplib2 for the underlying http where response.data is string instead of bytes.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label May 6, 2020
@arithmetic1728 arithmetic1728 requested a review from busunkim96 May 6, 2020 22:42
Copy link
Contributor

@busunkim96 busunkim96 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would you mind changing this to a fix since it changes the implementation?

@arithmetic1728 arithmetic1728 merged commit e115bae into master May 6, 2020
@arithmetic1728 arithmetic1728 deleted the sijun/gcloud branch May 6, 2020 23:00
@arithmetic1728
Copy link
Contributor Author

Would you mind changing this to a fix since it changes the implementation?

Ah, sorry I didn't notice your comment.

@busunkim96
Copy link
Contributor

@arithmetic1728 That's alright! You won't see release-please open a PR (it does patch bumps for fix, minor bumps for feat, and major bumps for breaking changes).

If this is urgent you can open a new PR and add the following to the commit body:

Release-As: 1.14.2

@arithmetic1728
Copy link
Contributor Author

@busunkim96 gcloud has its own google-auth copy in g3, so I just need to update in github and pull it into gcloud. I think I don't need to release it this time. Thank you!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants