-
Notifications
You must be signed in to change notification settings - Fork 9
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
CI for this project? #7
Comments
This is a great idea! I'll work on this immediately |
@StabbyCutyou it's important to test on a variety of CI systems to ensure cross platform compatibility. I'd recommend at least 5 different ones to be safe |
That's a good point. We cannot allow a false positive on any one CI system Can you recommend other free CI systems, and I can add backlog tasks to add On Sep 26, 2016 6:17 AM, "Ian Chiles" notifications@github.com wrote:
|
I've integrated Travis into the build. If anyone has further suggestions, please continue to open issues. |
Hi, just wondering if you have any plans on adding CI to this. Here are a couple of popular options:
http://circleci.com/
https://travis-ci.org/
I think this will really help future contributors and is a great way to demonstrate to the open source community that -- despite the complexity of this package -- the testing story is well looked after.
Thank you.
The text was updated successfully, but these errors were encountered: