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.

Sign up for GitHub

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

Jump to bottom

Adding connection lazy loading in storage. #822

Merged
merged 2 commits into from
Apr 15, 2015
Merged

Adding connection lazy loading in storage. #822

merged 2 commits into from
Apr 15, 2015

Conversation

dhermes
Copy link
Contributor

@dhermes dhermes commented Apr 13, 2015

Also note that gcloud/pubsub/test__implicit_environ.py was testing a storage function, not a pubsub one origin.

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

Coverage Status

Coverage remained the same at 100.0% when pulling 05dfb97 on dhermes:storage-lazy-loaded-connection into 4d2c187 on GoogleCloudPlatform:master.

This was referenced Apr 14, 2015
Merged
Closed
@tseaver
Copy link
Contributor

tseaver commented Apr 15, 2015

LGTM

dhermes added a commit that referenced this pull request Apr 15, 2015
@dhermes
Merge pull request #822 from dhermes/storage-lazy-loaded-connection
21395ef 
Adding connection lazy loading in storage.
@dhermes dhermes merged commit 21395ef into googleapis:master Apr 15, 2015
@dhermes dhermes deleted the storage-lazy-loaded-connection branch April 15, 2015 21:39
@dhermes dhermes mentioned this pull request Jun 4, 2015
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
api: storage Issues related to the Cloud Storage API. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@dhermes @coveralls @tseaver @googlebot