-
Notifications
You must be signed in to change notification settings - Fork 241
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
ENH: Adding travis.yml file #41
base: master
Are you sure you want to change the base?
Conversation
Cool - happy to give you commit rights (have invited as a collaborator) |
Seems that i need to be part of the organization to do this - wanna sync travis yourself and then merge this and see if it works? You may need to set up a PyFlux organisation or something to do that... |
Ah damn - sure, might have to look to move this repo out of my personal account. In meantime I'll try merging at weekend; the only thing I might have to fiddle with is the requirements. |
Np :) On Fri, Sep 9, 2016 at 1:32 PM, Ross Taylor [email protected]
Peadar Coyle |
Update - am intending to get Travis set up once I have written all the planned tests this week. Much excite... |
Cool :) On Mon, Nov 7, 2016 at 3:03 PM, Ross Taylor [email protected]
Peadar Coyle |
Nearly ready - just need to iron out the travis.yml file. Current issue is numdifftools on conda, |
How's this looking? |
Small issue with the py.test command, can't seem to import ._check_build |
In latest output of Travis CI it looks like there is another dependency missing for coverage with py.test, which can be fixed with pip install pytest-cov or by disabling coverage reports in this first setup... |
Thanks - I've been meaning to find some time to sit down and get this set up. Will include that dependency |
Finally found the issue - the issue is with _check_build folder which is bugging out the testing when called in init in the parent folder (as it is not actually contained with the package). Simply need to fix this import then should be good to go |
Opening a PR - you need to do this Ross or give me commit rights. https://ilovesymposia.com/2014/10/15/continuous-integration-in-python-4-set-up-travis-ci/