Skip to content

Conversation

ghost
Copy link

@ghost ghost commented May 15, 2015

No description provided.

@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project, in which case you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


  • If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
  • If you signed the CLA as a corporation, please let us know the company's name.

@googlebot googlebot added the cla: no This human has *not* signed the Contributor License Agreement. label May 15, 2015
@ghost
Copy link
Author

ghost commented May 15, 2015

I signed it!

@googlebot
Copy link

CLAs look good, thanks!

@googlebot googlebot added cla: yes This human has signed the Contributor License Agreement. and removed cla: no This human has *not* signed the Contributor License Agreement. labels May 15, 2015
@dhermes
Copy link
Contributor

dhermes commented May 15, 2015

Looks fine. Any reason why you don't just use the Blob.upload_* methods?

@ghost
Copy link
Author

ghost commented May 15, 2015

not particularly, either works

@dhermes
Copy link
Contributor

dhermes commented May 15, 2015

I think we've even debated making Bucket.upload_file a protected method to just encourage people to use the Blob directly.


Can you address the test failure:

TypeError: upload_from_filename() got an unexpected keyword argument 'content_type'

@ghost
Copy link
Author

ghost commented May 15, 2015

hm.. the test passes for me
I'll look into it tomorrow

@dhermes
Copy link
Contributor

dhermes commented May 15, 2015

Thanks. You are running the tests locally via tox, yes?

@ghost
Copy link
Author

ghost commented May 15, 2015

yes, I am also using the code in production

@dhermes
Copy link
Contributor

dhermes commented May 15, 2015

Notice it's a test fixture _Blob which encounters an unexpected keyword argument 'content_type': https://github.com/GoogleCloudPlatform/gcloud-python/blob/9c4867ab3a26b6f5e19f85c5d66e3c9083a8d83f/gcloud/storage/test_bucket.py#L525-L533

@dhermes dhermes closed this Aug 6, 2015
parthea pushed a commit that referenced this pull request Aug 21, 2025
* chore(python): fix typo in README

Source-Link: googleapis/synthtool@93e1685
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:631b4a35a4f9dd5e97740a97c4c117646eb85b35e103844dc49d152bd18694cd

* 🦉 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>
Co-authored-by: Lingqing Gan <lingqing.gan@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in 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