Skip to content
This repository has been archived by the owner on Nov 5, 2019. It is now read-only.

Adding Python 3.5 capability #531

Merged
merged 1 commit into from
Jun 29, 2016
Merged

Conversation

pferate
Copy link
Contributor

@pferate pferate commented Jun 28, 2016

In Python 3.5, http.HTTPStatus was added, previously HTTP statuses were int.
Casting http.HTTPStatus as int to maintain backwards maintainability.

@@ -1,6 +1,12 @@
language: python
python: 2.7
sudo: false
# A hack to install 3.5, until Travis-CI includes it

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

@nathanielmanistaatgoogle
Copy link
Contributor

@pferate
Copy link
Contributor Author

pferate commented Jun 28, 2016

I was using 80 characters, since it looked like a few previous commits were using that. I'll switch it to 72.

I'll also add an issue, then reference it in the .travis.yml file.

@@ -151,13 +151,13 @@ def test_token_refresh_store_expires_soon(self):
access_token = '1/3w'
token_response = {'access_token': access_token, 'expires_in': 3600}
http = HttpMockSequence([
({'status': str(http_client.UNAUTHORIZED)},
({'status': str(int(http_client.UNAUTHORIZED))},

This comment was marked as spam.

This comment was marked as spam.

In Python 3.5, `http.HTTPStatus` was added, previously HTTP statuses
were int.  Casting `http.HTTPStatus` as int to maintain backwards
maintainability.
@pferate
Copy link
Contributor Author

pferate commented Jun 28, 2016

Changes made, just waiting for Travis.

@theacodes
Copy link
Contributor

Thanks, this looks good to merge once travis passes.

@theacodes theacodes merged commit 61c61a0 into googleapis:master Jun 29, 2016
@pferate pferate deleted the python35 branch June 30, 2016 19:59
@theacodes theacodes mentioned this pull request Jul 28, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants