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 .travis.yml, build using Python 3.7-dev instead of 3.5 #727

Merged
merged 1 commit into from
Jul 10, 2018

Conversation

Mariatta
Copy link
Member

No description provided.

@@ -1,6 +1,6 @@
language: python
python:
- 3.5
- "3.7-dev"
Copy link
Member

Choose a reason for hiding this comment

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

Why dev and not just 3.7?

Copy link
Member Author

Choose a reason for hiding this comment

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

Living on the edge! 😄
I just copy 🍝 from core-workflow, bedevere, and the-knights-who-say-ni, all of them using "-dev".

Copy link
Member

Choose a reason for hiding this comment

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

I suppose those were written before 3.7.0 was released. I don't think it matters a lot but it seems safer to use the released version. :)

Copy link
Member Author

Choose a reason for hiding this comment

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

3.7 on travis is only available using dist: xenial, and it is not yet officially documented here.

Copy link
Member

Choose a reason for hiding this comment

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

Makes sense, thanks for looking into this!

Copy link
Member

@pradyunsg pradyunsg Jul 10, 2018

Choose a reason for hiding this comment

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

FWIW, it seems the 3.7-dev version is actually a beta or something similar and not the CPython's current master branch.

None the less, IIRC, we didn't have any major changes made in the rc period anyway. :)

Source: travis-ci/travis-ci#9815 (comment) -- that's also the issue where the xenial support was mentioned

@Mariatta Mariatta merged commit 765dc66 into master Jul 10, 2018
@Mariatta Mariatta deleted the Mariatta-patch-1 branch July 10, 2018 18:04
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.

4 participants