-
Notifications
You must be signed in to change notification settings - Fork 7
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
Implement continuous integration on Travis #37
Comments
@whikloj great idea, I will add that. |
@mjordan I have a simple travis setup in my branch for #35 to ensure it works. Currently its failing on code sniffer. So I might have some of those fixes too. |
I can't seem to run your tests, is there something I might need to do first? |
What is the output from |
Should phpunit be a dev requirement of the riprap? Instead I was running it using my installed phpunit, which could be the problem.
|
Okay I did some reading and now I see I should be running
which installs phpunit and seems to result in the same errors as above. |
Guessing it would be good to make sure this builds in multiple PHP versions. A simple travis matrix would allow it to check that it installs and that your tests pass.
The text was updated successfully, but these errors were encountered: