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

Coverage for 'gcloud._apitools.http_wrapper' #1196

Merged
merged 13 commits into from
Oct 26, 2015
Merged

Coverage for 'gcloud._apitools.http_wrapper' #1196

merged 13 commits into from
Oct 26, 2015

Conversation

tseaver
Copy link
Contributor

@tseaver tseaver commented Oct 20, 2015

Uses #1193 as a base.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 20, 2015
@tseaver
Copy link
Contributor Author

tseaver commented Oct 26, 2015

@dhermes rebased after merge of #1193.

@@ -91,8 +90,7 @@ def _Httplib2Debuglevel(http_request, level, http=None):
httplib2.debuglevel = old_level
if http is not None:
for connection_key, old_level in http_levels.items():
if connection_key in http.connections:
http.connections[connection_key].set_debuglevel(old_level)
http.connections[connection_key].set_debuglevel(old_level)

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

@dhermes
Copy link
Contributor

dhermes commented Oct 26, 2015

LGTM

tseaver added a commit that referenced this pull request Oct 26, 2015
…wrapper

Coverage for 'gcloud._apitools.http_wrapper'
@tseaver tseaver merged commit 2ef1639 into googleapis:master Oct 26, 2015
@tseaver tseaver deleted the coverage-gcloud__apitools_http_wrapper branch October 26, 2015 19:23
parthea pushed a commit that referenced this pull request Oct 21, 2023
…leCloudPlatform/python-docs-samples#1196)

* Adds snippets for enabling and restoring a key version

* Fixed lint issues
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: core cla: yes This human has signed the Contributor License Agreement. testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants