Skip to content

Conversation

@richardevs
Copy link
Contributor

Remove deprecated function download_as_string(), replace with download_as_bytes(). [1]
[1] https://github.com/googleapis/python-storage/blob/master/google/cloud/storage/blob.py#L138

Fixes #504 🦕

@richardevs richardevs requested a review from a team July 15, 2021 15:12
@richardevs richardevs requested a review from a team as a code owner July 15, 2021 15:12
@product-auto-label product-auto-label bot added the api: storage Issues related to the googleapis/python-storage API. label Jul 15, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Jul 15, 2021
@tseaver
Copy link
Contributor

tseaver commented Jul 15, 2021

@richardevs Thanks for the patch! Next time, you can edit the PR title to fix the CC check, or do a git commit --amend followed by git push -f.

@tseaver tseaver added automerge Merge the pull request once unit tests and other checks pass. kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Jul 15, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 15, 2021
@cojenco cojenco added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 15, 2021
@gcf-merge-on-green
Copy link
Contributor

Merge-on-green attempted to merge your PR for 6 hours, but it was not mergeable because either one of your required status checks failed, one of your required reviews was not approved, or there is a do not merge label. Learn more about your required status checks here: https://help.github.com/en/github/administering-a-repository/enabling-required-status-checks. You can remove and reapply the label to re-run the bot.

@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Jul 16, 2021
@tseaver tseaver added kokoro:force-run Add this label to force Kokoro to re-run the tests. and removed kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Jul 16, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 16, 2021
@tseaver tseaver added the automerge Merge the pull request once unit tests and other checks pass. label Jul 16, 2021
@gcf-merge-on-green gcf-merge-on-green bot merged commit 8dfa4d4 into googleapis:master Jul 16, 2021
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Jul 16, 2021
cojenco pushed a commit to cojenco/python-storage that referenced this pull request Oct 13, 2021
cojenco pushed a commit to cojenco/python-storage that referenced this pull request Oct 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: storage Issues related to the googleapis/python-storage API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove deprecated function download_as_string() in Example Usage

4 participants