Skip to content

Conversation

@kernc
Copy link
Contributor

@kernc kernc commented Nov 18, 2016

Contrary to other CI attempts, I choose the least invested route that is hopefully very easy to review and merge.

This just runs the test examples on Travis, reporting any failures. When you enable CI for this repo on Travis, as you can see, the build passes, except for the newer Python versions which you will eventually (hopefully) bring up to speed. 😄

Python <=2.6 support is not tested. Nobody uses that any more.

@kernc
Copy link
Contributor Author

kernc commented Nov 25, 2016

@mmckerns How does this look to you? 😃 One need only enable the repo on Travis to see it working quite well.

@Peque Peque mentioned this pull request Dec 17, 2016
@mmckerns mmckerns mentioned this pull request Dec 21, 2016
@mmckerns
Copy link
Member

@kernc: This looks good. dill needs to be using CI. I wanted to be using it already a few years ago, and just have not gotten around to it. So I will review this and #198, and see if we can PR one or both.

@kernc kernc force-pushed the travis branch 3 times, most recently from a8df063 to 3170605 Compare December 26, 2016 21:26
@mmckerns mmckerns merged commit 600c8b1 into uqfoundation:master Dec 27, 2016
@kernc kernc deleted the travis branch December 27, 2016 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants