Skip to content

Updated travis configuration to use pytest #13

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

guyzmo
Copy link
Contributor

@guyzmo guyzmo commented May 8, 2017

changes are:

update to pytest
usage of latest pip version
usage of virtualenv
show coverage results as part of the log output

@guyzmo guyzmo force-pushed the features/pytest_travis branch from edbab2f to f804654 Compare May 8, 2017 18:33
@ethantkoenig
Copy link
Contributor

Why is this better than what we had previously?

@guyzmo
Copy link
Contributor Author

guyzmo commented May 9, 2017

  1. pytest is a neat framework to work with, it's a rather minor change as is (it's built so it's compatible with unittest, but provide more abilities for debugging, like -x break at first failed test, --lf repeat last failure), and a bunch of fixtures that can be useful for testing
  2. virtualenv + latest pip is to make sure that latest environment is running and is easily reproduceable
  3. show coverage is just so that you don't need to push to travis to see changes in coverage 😉

@ethantkoenig
Copy link
Contributor

@guyzmo Sorry for the delayed response; the last week has been busy.

I get why you might want to use pytest, virtualenv etc. locally, but it's not clear to me why our CI build needs to be updated.

@jag3773
Copy link
Contributor

jag3773 commented Sep 29, 2017

@guyzmo If you put these into a Makefile then that would make it easy to use locally.

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