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 configuration #44

Merged
merged 4 commits into from
Jan 16, 2020

Conversation

wisp3rwind
Copy link
Collaborator

@wisp3rwind wisp3rwind commented Jan 15, 2020

  • Bump python versions
  • travis appears to use Xenial everywhere nowadays, so no need to force this distribution in order to get recent Python
  • the sudo keyword was deprecated because travis now uses a fully virtualized infrastructure: link

@wisp3rwind wisp3rwind force-pushed the pr-update-travis-cfg branch 2 times, most recently from d2c36f5 to 046a9fe Compare January 15, 2020 22:49
@wisp3rwind
Copy link
Collaborator Author

Fails because nosetests is dead, and does not support Python 3.9, nose-devs/nose#1099

@wisp3rwind
Copy link
Collaborator Author

Tentatively, I replaced nose with pytest. I'm not super familiar with Python's testing ecosystem, so maybe that's not the best choice. There's at least also nose2. Any opinion on the test runner, @geigerzaehler?

Copy link
Owner

@geigerzaehler geigerzaehler left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this! pytest is fine with me, that is what I was using recently.

I think we may also want to update the envlist in tox.ini.

@wisp3rwind wisp3rwind force-pushed the pr-update-travis-cfg branch from 7c154b2 to e2f0422 Compare January 16, 2020 09:25
@wisp3rwind
Copy link
Collaborator Author

I think we may also want to update the envlist in tox.ini.

👍

Copy link
Owner

@geigerzaehler geigerzaehler left a comment

Choose a reason for hiding this comment

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

🚀 :shipit:

@wisp3rwind wisp3rwind merged commit 8cc37a9 into geigerzaehler:master Jan 16, 2020
@wisp3rwind wisp3rwind deleted the pr-update-travis-cfg branch January 16, 2020 18:38
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.

2 participants