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
Have AppVeyor run the tests too #1188
Conversation
Should be save to use |
Perhaps, you can't enable the test upload inside a PR, so we would have to merge this first. I think this is ready once, all the test commands are removed. |
Sure, we can merge this now if you like and see if the tests magically start working. At the very least, running the tests at all is an improvement. I can open an issue after this is merged about getting the test output to upload correctly. |
Sounds good. |
@mlpack-jenkins test this please |
Let's see what this does...