Migrate to GitHub Actions #2156
Conversation
[skip travis] This will eventually replace Travis Just two builds to check the matrix works as intended
[skip travis]
[skip travis]
[skip travis]
[skip travis]
[skip travis]
For some reason, installing python3-h5py broke running mpiexec from Python using subprocess in ubuntu-20.04. I do not understand why, but this was definitely the cause
I think the actions run on a PR are the ones in the |
|
Coverage job still not running on the PR build... think I'll need to investigate in a fork so as to not spam this repo! I've deleted the |
github.base_ref doesn't include 'refs/heads/' while github.ref does
|
clang-tidy review says "All clean, LGTM! 👍" |
|
🤦 Ignore all that, meant to push those debugging commits to my fork |
Comment on PRs with warnings from clang-tidy
ZedThree
force-pushed
the
migrate-github-actions
branch
from
41b12a6
to
3234067
Compare
|
Coverage job now working, I've also added a Assuming that all the tests now actually run and pass, this is ready to go in. As it gets merged into other branches, they'll switch over from Travis to GitHub Actions. I've set these tests to be required, but I think I'll have to undo that for the time being, otherwise it's going to be a little confusing as they won't run on other branches |
I've disabled Travis for the time being, rather than delete the
.travis.yml, as I want to check this runs the coverage tests on the PR correctly without Travis running at the same time.This is going into
next-- I'm not sure if the action becomes available in all branches.I've also tweaked the jobs a little bit, so they're not exactly the same as on Travis. Most are now running on ubuntu-18.04 (Travis is all 16.04), except the CMake one is on 20.04.
It doesn't seem to be any faster than Travis, which is a little bit of a shame.
EDIT: Looks like the coverage jobs aren't running, so that'll be a fun job for Monday.