-
Notifications
You must be signed in to change notification settings - Fork 263
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
Bump python versions up to 3.6 and 3.7 #1235
Conversation
7512b05
to
01a34e7
Compare
.travis.yml
Outdated
language: python | ||
python: | ||
- 3.5 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unless there's anything backwards incompatible with 3.5 that we'd be looking to include I'd prefer to keep it (and of course add 3.7).
(For example, on a purely selfish note, for my teaching I'd need to get anaconda reinstalled on all our lab machines if we dropped 3.5.)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ignore this I've just seen #1233 (comment)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm fine with keeping 3.5 if you need it. We just have to use the comment form of type hint, which we've already done in some cases.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If it's easy enough it could be nicer to keep things a bit more backwards compatible but I'm happy with whatever you prefer.
@@ -23,15 +23,15 @@ | |||
packages=["axelrod", "axelrod.strategies", "axelrod.data"], | |||
url="http://axelrod.readthedocs.org/", | |||
license="The MIT License (MIT)", | |||
description="Reproduce the Axelrod iterated prisoners dilemma tournament", | |||
description="Axelrod game theory library for the iterated prisoners dilemma", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
README.rst
Outdated
@@ -74,7 +74,8 @@ a peer reviewed paper introducing the library (22 authors). | |||
Installation | |||
------------ | |||
|
|||
The library requires Python 3.5 or greater. | |||
The library requires Python 3.6 or greater. We typically support the two latest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I know we said that we'd support the two latest in the past but perhaps we keep previous versions until there's a particular feature we want to take advantage of? (Perhaps there already is and I'm forgetting.)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've just seen #1233 (comment) which seems like a good reason to me. Would it be ok to just bump here to 3.6 and not include the We typically support the two latest...
?
I'm trying 3.5 again. I had to bump up to xenial for Travis to get 3.7, we'll see if it gets 3.5 still. |
Looks like travis isn't immediately happy, if it's a choice between 3.5 and 3.7 I'd suggest moving forward with 3.7 :) |
Well I haven't figured out the necessary voodoo to make this work with both 3.5 and 3.7 at the same time, so I'll close for now until we decide to drop 3.5. |
No description provided.