-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
test: upgrade Linux Travis CI to Python 3.8 #1923
Conversation
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.
Thanks @cclauss!
@cclauss the changes looked ok to me, but travis hit
|
Yes. That fix was landed in #1921. Can someone with Travis admin rights please rerun the tests? |
Ensure that we remain compatible with the newest version of Python.
a069872
to
794b6eb
Compare
@cclauss I've rebased to make it run. But it looks like there's a lot of backlog in Travis... |
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.
+1 if this passes, it didn't yesterday when I tried but maybe they've released new tarballs for it?
Ensure that we remain compatible with the newest version of Python. PR-URL: #1923 Reviewed-By: Rod Vagg <rod@vagg.org> Reviewed-By: Sam Roberts <vieuxtech@gmail.com> Reviewed-By: João Reis <reis@janeasystems.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com>
landed in b91718e |
Ensure that we remain compatible with the newest version of Python. PR-URL: #1923 Reviewed-By: Rod Vagg <rod@vagg.org> Reviewed-By: Sam Roberts <vieuxtech@gmail.com> Reviewed-By: João Reis <reis@janeasystems.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Blocked by #1921
Ensure that we remain compatible with the newest version of Python.
Checklist
npm install && npm test
passesDescription of change