Skip to content
This repository was archived by the owner on Dec 10, 2025. It is now read-only.

Conversation

@dhermes
Copy link
Contributor

@dhermes dhermes commented Sep 29, 2016

In the process, also renaming all instances of gcloud.

NOTE: Has #10 as diffbase.

'google-cloud-bigtable >= 0.20.0',
]

SETUP_BASE.pop('url')

This comment was marked as spam.

Copy link

@theacodes theacodes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This LGTM once travis passes.

In the process, also renaming all instances of gcloud.
In order to get passing system tests, Client.start()
and Client.stop() had to stop being used.

Also add a newline (per feedback) in setup.py.
@theacodes
Copy link

New changes in 7369de1 also LGTM

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 7369de1 on dhermes:upgrade-deps into * on GoogleCloudPlatform:master*.

@dhermes dhermes merged commit 1a81d39 into googleapis:master Sep 29, 2016
@dhermes dhermes deleted the upgrade-deps branch September 29, 2016 18:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants