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

Travis CI basic configuration #24

Merged
merged 4 commits into from
Aug 20, 2018

Conversation

mattrossman
Copy link
Collaborator

See #18

Adds dummy test and a simple search test. Fixes import of dining_utils.
@mattrossman
Copy link
Collaborator Author

To test locally, install pytest with

pip install pytest

and run

pytest

from the project root. Both tests should pass.

@mattrossman
Copy link
Collaborator Author

mattrossman commented Aug 20, 2018

I'm not totally clear on how the default Travis CI environment is set up, but from what I understand it has access to pytest automatically. travis-ci.yml might need some tweaks to get the environment properly set up but we can see what happens after a test build.

EDIT: I'm a idiot, should be .travis.yml not travis-ci.yml

@mattrossman
Copy link
Collaborator Author

Build is passing, should be good to merge

@simon-andrews simon-andrews merged commit 0cc5847 into simon-andrews:master Aug 20, 2018
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