Skip to content

Remove Bucket.new_blob - #725

Merged
dhermes merged 2 commits into
googleapis:masterfrom
dhermes:remove-new-blob
Mar 17, 2015
Merged

Remove Bucket.new_blob#725
dhermes merged 2 commits into
googleapis:masterfrom
dhermes:remove-new-blob

Conversation

@dhermes

Copy link
Copy Markdown
Contributor

Was not necessary, Blob() constructor suffices.

@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 13, 2015
Updated Bucket.get_blob() and Bucket.delete_blob() to just take
a string instead of string or Blob. Logic being that if the user
already has a Blob, they can just call blob.delete() or
blob._reload_properties() (more convenient method needed).
Updated Bucket.delete_blobs() to check if each element in list of `blobs`
is a string or a Blob.
Updated Blob.delete() to call Bucket.delete_blob() with just a string and
updated Blob.rename() to call Blob.delete() instead of re-implementing the
mapping to Bucket.delete_blob().
Removed Test_Bucket.test_new_blob_existing, the last caller of
Bucket.new_blob that sends a Blob instead of a string.
Opting to just use Blob() constructor where relevant.
@dhermesdhermes added the api: storage Issues related to the Cloud Storage API. label Mar 13, 2015
@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 100.0% when pulling d981af6 on dhermes:remove-new-blob into d668044 on GoogleCloudPlatform:master.

@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 100.0% when pulling d981af6 on dhermes:remove-new-blob into d668044 on GoogleCloudPlatform:master.

@tseaver

Copy link
Copy Markdown
Contributor

LGTM

dhermes added a commit that referenced this pull request Mar 17, 2015
@dhermes
dhermes merged commit 6d147c8 into googleapis:masterMar 17, 2015
@dhermes
dhermes deleted the remove-new-blob branch March 17, 2015 02:56
dhermes added a commit to dhermes/google-cloud-python that referenced this pull request Mar 17, 2015
dhermes added a commit that referenced this pull request Mar 17, 2015
@dhermesdhermes mentioned this pull request Apr 9, 2015
parthea pushed a commit that referenced this pull request Aug 21, 2025
* chore(deps): update all dependencies
* 🦉 Updates from OwlBot post-processor
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
* pin ipython version for python 3.9
because later versions dropped support for python 3.9
* pin pytest for python 3.7
* pin pytest for python 3.7
* pin pytest for python 3.7
---------
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Co-authored-by: Lingqing Gan <lingqing.gan@gmail.com>
parthea pushed a commit that referenced this pull request Sep 16, 2025
* chore(deps): update all dependencies
* 🦉 Updates from OwlBot post-processor
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
* pin ipython version for python 3.9
because later versions dropped support for python 3.9
* pin pytest for python 3.7
* pin pytest for python 3.7
* pin pytest for python 3.7
---------
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Co-authored-by: Lingqing Gan <lingqing.gan@gmail.com>
parthea pushed a commit that referenced this pull request Sep 18, 2025
Source-Link: https://togithub.com/googleapis/synthtool/commit/fac8444edd5f5526e804c306b766a271772a3e2f
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:5ea6d0ab82c956b50962f91d94e206d3921537ae5fe1549ec5326381d8905cfa
parthea pushed a commit that referenced this pull request Nov 22, 2025
* docs: Add documentation for enums
fix: Add context manager return types
chore: Update gapic-generator-python to v1.8.1
PiperOrigin-RevId: 503210727
Source-Link: googleapis/googleapis@a391fd1
Source-Link: googleapis/googleapis-gen@0080f83
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiMDA4MGY4MzBkZWMzN2MzMzg0MTU3MDgyYmNlMjc5ZTM3MDc5ZWE1OCJ9
* 🦉 Updates from OwlBot post-processor
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
parthea pushed a commit that referenced this pull request Nov 24, 2025
…p/templates/python_library/.kokoro (#725)
Source-Link: googleapis/synthtool@d0f51a0
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:240b5bcc2bafd450912d2da2be15e62bc6de2cf839823ae4bf94d4f392b451dc
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
parthea pushed a commit that referenced this pull request Nov 24, 2025
Both dropped Python2 support in a minor release. grpcio does not declare
'python_requires'.
parthea pushed a commit that referenced this pull request Mar 6, 2026
parthea pushed a commit that referenced this pull request Mar 6, 2026
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
parthea pushed a commit that referenced this pull request Mar 9, 2026
#725)
@ddelgrosso1 Related to b/217259317.
Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:
- [ ] Make sure to open an issue as a [bug/issue](https://github.com/googleapis/python-storage/issues/new/choose) before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
- [ ] Ensure the tests and linter pass
- [ ] Code coverage does not decrease (if any source code was changed)
- [ ] Appropriate docs were updated (if necessary)
Fixes #<issue_number_goes_here> 🦕
parthea pushed a commit that referenced this pull request Mar 9, 2026
* chore: use gapic-generator-python 0.65.2
PiperOrigin-RevId: 444333013
Source-Link: googleapis/googleapis@f91b6cf
Source-Link: googleapis/googleapis-gen@16eb360
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiMTZlYjM2MDk1YzI5NGU3MTJjNzRhMWJmMjM1NTA4MTdiNDIxNzRlNSJ9
* 🦉 Updates from OwlBot post-processor
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: storageIssues related to the Cloud Storage API.cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@dhermes@coveralls@tseaver@googlebot