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

Please add Travis CI integration #42

Closed
sschuberth opened this issue Nov 26, 2017 · 4 comments
Closed

Please add Travis CI integration #42

sschuberth opened this issue Nov 26, 2017 · 4 comments

Comments

@sschuberth
Copy link
Contributor

To ensure PRs pass the tests.

@ccavanaugh
Copy link
Owner

https://travis-ci.org/ccavanaugh/jgnash

Basics for Travis are enable, but it's puking when building the Windows EXE's

Any ideas? I can see value in this, but not if it's too intrusive.

@sschuberth
Copy link
Contributor Author

sschuberth commented Nov 28, 2017

Travis does not support Windows, only Linux and macOS. For Windows CI you'd need to use e.g. AppVeyor. As an example, you may want to look at this project that integrates with both.

@ccavanaugh
Copy link
Owner

The windows exe's are still a Linux based app / Gradle plugin

@ccavanaugh
Copy link
Owner

Working now.... and it did expose a unit test failure I had not found since the last release build.

The fix for building the windows EXE's was to install some lib32 dependencies.

Still learning the full capability, so open to suggestions for improvement.

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

No branches or pull requests

2 participants