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

Update supported Python versions #170

Merged
merged 4 commits into from
Jun 7, 2017
Merged

Update supported Python versions #170

merged 4 commits into from
Jun 7, 2017

Conversation

scasagrande
Copy link
Contributor

@scasagrande scasagrande commented Jun 6, 2017

This PR adds support for Python 3.6 and drops support for 3.3

The reason for dropping 3.3 support is because numpy>=1.12 doesn't support Py33. Since numpy is currently still a requirement, I think its important that we don't try to downgrade (or change) our users' version of numpy when IK doesn't actually need to.

Related issue: #168

@coveralls
Copy link

coveralls commented Jun 6, 2017

Coverage Status

Coverage remained the same at 84.956% when pulling 73b2c17 on update_py_versions into 6c6dbb4 on develop.

@scasagrande
Copy link
Contributor Author

This shouldn't impact you, but just tagging just in case @cgranade @crazy4pi314

@cgranade
Copy link
Contributor

cgranade commented Jun 7, 2017

I think that makes a lot of sense. Unlike 2.7 → 3.x, there's very little pain in upgrading 3.3 → 3.4, such that putting 3.4 as a minimum requirement for 3.x support seems entirely reasonable. Hopefully we can even take the leap to 3.5 and no 2.7 at some point, so that we can get that sweet async/await logic going.

@scasagrande
Copy link
Contributor Author

I'm kinda thinking that 2017 might be the last year for 2.7 support here. I'll bring up the discussion in an issue at some point.

@scasagrande scasagrande merged commit 3316327 into develop Jun 7, 2017
@scasagrande scasagrande deleted the update_py_versions branch June 7, 2017 03:16
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.

3 participants