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

Python 2.6, 2.7, 3.4, 3.5, 3.6, 3.7 compatibility #395

Closed
wants to merge 1 commit into from

Conversation

xavierhardy
Copy link

This pull request brings compatibility from Python 3.4 to Python 3.7 included (await is keyword in 3.7).

@xavierhardy
Copy link
Author

Related to issue #218

@xavierhardy xavierhardy force-pushed the python3 branch 2 times, most recently from 41eca80 to 18a1d13 Compare July 22, 2018 16:26
@xavierhardy xavierhardy changed the title Python 2.7, 3.4, 3.5, 3.6, 3.7 compatibility Python 2.6, 2.7, 3.4, 3.5, 3.6, 3.7 compatibility Jul 22, 2018
@xavierhardy xavierhardy force-pushed the python3 branch 3 times, most recently from 470ea39 to 16f7a2f Compare July 22, 2018 17:58
@johann8384
Copy link
Member

This is great! I did some work on this as well, and did some refactoring to make testing easier. I'll rebase mine to work off this, you went through more collectors than I had gotten to.

@johann8384
Copy link
Member

Can you add the additional Python version to the Travis build in your branch? Then we can see that is passes with all the versions for your PR.

https://github.com/OpenTSDB/tcollector/blob/master/.travis.yml#L3

Thank you !!

@johann8384
Copy link
Member

Merged in #411

@johann8384 johann8384 closed this Dec 4, 2018
@xavierhardy
Copy link
Author

Thanks for merging and adding the Python 3 environments to Travis.

@johann8384 johann8384 added this to the 1.3.3 milestone Dec 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants