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

Do test on Python 3.7 #178

Merged
merged 2 commits into from Nov 6, 2018
Merged

Do test on Python 3.7 #178

merged 2 commits into from Nov 6, 2018

Conversation

Freso
Copy link
Member

@Freso Freso commented Nov 5, 2018

Picard supports Python 3.7, so making sure plugins work on it is very useful.

Heavily inspired by #177 ;)

Picard supports Python 3.7, so making sure plugins work on it is
very useful.

Heavily inspired by
metabrainz#177 ;)
@Freso
Copy link
Member Author

Freso commented Nov 5, 2018

Ugh. Looks like the Travisbuntu is too ancient to have Py3.7? I'll try and look into this later.

@phw
Copy link
Member

phw commented Nov 5, 2018

Unfortunately it won't work that easily, unless they have Python 3.7 finally available on Travis CI :) See metabrainz/picard@c0448cb for a working solution if the builds will fail

EDIT: Yep, failed :(

Ubuntu 14.04 Trusty (Travis' default distribution) doesn't contain
Python 3.7 and is generally more than 4 years old and I think not
supported by Picard 2.0 anyway, so moving to a 2 years never version of
Ubuntu that does contain Python 3.7.
@Freso
Copy link
Member Author

Freso commented Nov 6, 2018

Instead of making an additional matrix entry, I just changed the distribution used to be Ubuntu 16.04 instead of 14.04.

@phw phw merged commit 5881023 into metabrainz:2.0 Nov 6, 2018
@Freso Freso deleted the yes-python37 branch November 6, 2018 14:22
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.

None yet

3 participants