Skip to content
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

Making name= the first argument in Bucket() constructor. #737

Merged
merged 1 commit into from
Mar 17, 2015

Conversation

dhermes
Copy link
Contributor

@dhermes dhermes commented Mar 14, 2015

This is to allow Bucket('foo') in the implicit connection case, which we want to be an easy to use default for most.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 14, 2015
@dhermes dhermes added the api: storage Issues related to the Cloud Storage API. label Mar 14, 2015
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 7800ed5 on dhermes:bucket-make-name-first into d668044 on GoogleCloudPlatform:master.

@tseaver
Copy link
Contributor

tseaver commented Mar 16, 2015

LGTM

This is to allow Bucket('foo') in the implicit connection case,
which we want to be an easy to use default for most.
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 453c0bf on dhermes:bucket-make-name-first into * on GoogleCloudPlatform:master*.

dhermes added a commit that referenced this pull request Mar 17, 2015
Making name= the first argument in Bucket() constructor.
@dhermes dhermes merged commit aa684be into googleapis:master Mar 17, 2015
@dhermes dhermes deleted the bucket-make-name-first branch March 17, 2015 22:05
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 Cloud 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.

4 participants