Skip to content

Conversation

@rayluo
Copy link
Contributor

@rayluo rayluo commented Oct 15, 2019

Python 3.8.0 has been released yesterday. Now we include it in our test automation matrix.

@rayluo rayluo requested a review from abhidnya13 October 15, 2019 19:23
# Enable 3.7 without globally enabling sudo and dist: xenial for other build jobs
matrix:
include:
- python: 3.7

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • python: 3.7 [](start = 4, length = 13)

remove ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, we don't want to remove any of the previous versions. We want the test runs to cover all major Python versions, as seen here.

@henrik-me
Copy link

henrik-me commented Oct 15, 2019

  • "3.6"

does new options need to be added here? If so are those needed in the include section?


Refers to: .travis.yml:6 in eb80b95. [](commit_id = eb80b95, deletion_comment = False)


Answered inline to keep Q&A stay together:

It is indeed a weird thing that this test infrastructure requires the newer Python versions (3.7, 3.8) to be configured in a different way. But again, as long as it covers all Python versions that we need, it is fine.

Copy link

@henrik-me henrik-me left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@rayluo rayluo merged commit d640457 into dev Oct 15, 2019
@rayluo rayluo deleted the python38 branch October 15, 2019 21:53
@rayluo rayluo mentioned this pull request Oct 18, 2019
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