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

ENH: Adding travis.yml file #41

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

Conversation

springcoil
Copy link
Collaborator

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/

@RJT1990
Copy link
Owner

RJT1990 commented Sep 9, 2016

Cool - happy to give you commit rights (have invited as a collaborator)

@springcoil
Copy link
Collaborator Author

springcoil commented Sep 9, 2016

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...

@RJT1990
Copy link
Owner

RJT1990 commented Sep 9, 2016

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.

@springcoil
Copy link
Collaborator Author

Np :)

On Fri, Sep 9, 2016 at 1:32 PM, Ross Taylor [email protected]
wrote:

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.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#41 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA8DiB4MTUz48BTau38aspk0DRoTuNPdks5qoVHAgaJpZM4J2Dio
.

Peadar Coyle
Skype: springcoilarch
www.twitter.com/springcoil
peadarcoyle.wordpress.com

@RJT1990
Copy link
Owner

RJT1990 commented Nov 7, 2016

Update - am intending to get Travis set up once I have written all the planned tests this week. Much excite...

@springcoil
Copy link
Collaborator Author

Cool :)

On Mon, Nov 7, 2016 at 3:03 PM, Ross Taylor [email protected]
wrote:

Update - am intending to get Travis set up once I have written all the
planned tests this week. Much excite...


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#41 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA8DiLyFmGsl-FjXErfJrwAgreH_Qfpxks5q7z2sgaJpZM4J2Dio
.

Peadar Coyle
Skype: springcoilarch
www.twitter.com/springcoil
peadarcoyle.wordpress.com

@RJT1990
Copy link
Owner

RJT1990 commented Nov 26, 2016

Nearly ready - just need to iron out the travis.yml file. Current issue is numdifftools on conda,

@springcoil
Copy link
Collaborator Author

How's this looking?

@RJT1990
Copy link
Owner

RJT1990 commented Dec 1, 2016

Small issue with the py.test command, can't seem to import ._check_build

@rth
Copy link

rth commented Jan 9, 2017

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...

@RJT1990
Copy link
Owner

RJT1990 commented Jan 11, 2017

Thanks - I've been meaning to find some time to sit down and get this set up. Will include that dependency

@RJT1990
Copy link
Owner

RJT1990 commented Jan 16, 2017

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

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