-
Notifications
You must be signed in to change notification settings - Fork 32
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
Move tests from Travis to GitHub Actions to test on all OSses #68
Comments
Hum... Even with matlab and octave? I know that Travis actually now supports Matlab in their CI so we could have both there. Not sure about github actions. |
Octave yes (not sure about Windows to be honest) - but Matlab probably not. I didnt know that Travis supports Matlab in their CI, that's awesome ... and a bit puzzling to me how they deal with the licensing issues :) |
Well I don't know much more than that: https://twitter.com/RemiGau/status/1310681042714537991 |
I now found this: https://github.com/joergbrech/moxunit-action but haven't tested it yet.
Looks like we need some time and energy to test a couple of alternatives :-) |
Ha yeah we started thinking about using MOxUnit in #44 . No decision has been made about this yet. But it definitely is an option that can also get you your code coverage in one fell swoop. |
any pointer on trying to adapt our github action workflows to run our tests also on MacOS and Windows (for octave)? I don't have the bandwidth to look on setting up CI tests with matlab at the moment. |
Yes, I think setting up for at least MacOS will be relatively straight forward: You will just have to make another step like this one: bids-matlab/.github/workflows/run_tests.yml Lines 22 to 26 in 6c1dac6
but ensure that the (then two) steps run depending on the OS. that is, one runs only when OS=ubuntu, the other one only when OS=macOs For the Ubuntu step, you leave everything as it is ... and for the MacOS step you adjust the "apt" installs with "brew" installs. The remaining action should work regardless of ubuntu or MacOS. --> for windows it would go the same way ... first duplicating the steps, then making them run conditionally on a single OS only, and then adjusting the duplicated-OS-specific steps to actually work on that OS. I mean -> apt on ubuntu, brew on macOS, and maybe chocolatey on windows, ... and so on for all other commands. I am not sure how easy this will be with windows. I once managed it for a similarly "complicated" install for the bids-validator. Hopefully it can serve as an example of what I mean: |
Awesome. Thanks for all those pointers. Moving this to a separate issue. 😉 |
HI all, This seems relevant: https://github.com/matlab-actions They are not in the marketplace yet, but they work just as the travis, circle, and azure DevOps extensions. Have fun! |
GitHub Actions allows for easy testing on Windows, MacOS, and Linux, which would improve the testing pipeline to also cover "special" bugs that only appear on some OSses.
The text was updated successfully, but these errors were encountered: