Closed agitter closed 3 years ago
The core SINGE tests are working, but this does not yet push the image to DockerHub or check the md5sum of the source files. The md5sums of parseParams.m
did not match.
The Docker image must be pulled from Docker Hub before building in order to use the cache. This works as of 37cfa55.
The behavior is slightly different from the old Travis CI workflow. The workflow should only push the image on the master branch, like Travis. However, it pushes before running the test cases. Potentially a broken image could be pushed. If we continue to use the pull request model, this should not be an issue. Failed test cases will be caught in the pull request so that they pass before we merge to master. I'm okay with this new behavior but want to document the change and am leaving .travis.yml
for now.
I'll restore the md5sum check in a separate pull request after recompiling the executables.
Travis CI now requires paid credits https://blog.travis-ci.com/2020-11-02-travis-ci-new-billing