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

Bump python versions up to 3.6 and 3.7 #1235

Closed
wants to merge 1 commit into from
Closed

Bump python versions up to 3.6 and 3.7 #1235

wants to merge 1 commit into from

Conversation

marcharper
Copy link
Member

No description provided.

.travis.yml Outdated
language: python
python:
- 3.5
Copy link
Member

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.)

Copy link
Member

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)

Copy link
Member Author

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.

Copy link
Member

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",
Copy link
Member

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
Copy link
Member

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.)

Copy link
Member

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...?

@marcharper
Copy link
Member Author

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.

@drvinceknight
Copy link
Member

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 :)

@marcharper
Copy link
Member Author

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.

@marcharper marcharper closed this Jan 15, 2019
@marcharper marcharper deleted the py37 branch September 3, 2019 06:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants