Skip to content
This repository has been archived by the owner on Apr 16, 2023. It is now read-only.

Added Travis and Appveyor setup (fixes #1) #62

Merged
merged 4 commits into from
Dec 28, 2018
Merged

Added Travis and Appveyor setup (fixes #1) #62

merged 4 commits into from
Dec 28, 2018

Conversation

jameslamb
Copy link
Collaborator

In this PR, I attempted to add a basic Travis CI setup. This will build your package + run the tests on:

  • every commit on an open PR
  • every merge to master

I had to make a slight modification to setup.py, declaring the testing packages needed.

Let's see if it builds, and if it does I'll add more Python versions and mac OS to this PR.

@jameslamb jameslamb changed the title Added Travis setup (fixes #1) Added Travis and Appveyor setup (fixes #1) Dec 28, 2018
@jameslamb
Copy link
Collaborator Author

Ok @gsganden I did set you up on autofocus but I'm pulling it out of this PR. I was getting failures there but not Travis which suggests Windows-specific problems. Since you and all the current users of this are on Macs or linux machines, it's not worth the administrative burden.

Created #63 to capture the work.

Copy link
Collaborator

@gsganden gsganden left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, thanks!

@gsganden gsganden merged commit 144a707 into uptake:master Dec 28, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants