Merge pull request #34 from rem1776/CI-updates

CI container update and fix for log files

.github/workflows/main.yml

@@ -1,28 +1,35 @@
 # Github actions CI file
 # Ryan Mulhall 2020
-# Builds and runs make check in two containers, one with mpi, one without
+# CI testing for the FRE-NCtools repo, builds and runs unit tests
+# container @ https://hub.docker.com/repository/docker/ryanmulhall/fre-nctools-base
 name: FRE-NCtools CI
 on: [push, pull_request]
 jobs:
-  build:
+  CI:
     runs-on: ubuntu-latest
     strategy:
       matrix:
         with_mpi: ['','--with-mpi']
         enable_quad_precision: ['', '--enable-quad-precision']
-    env:
+    container:
+      image: ryanmulhall/fre-nctools-base
+      env:
         MPI: ${{ matrix.with_mpi }}
         QUAD_P: ${{ matrix.enable_quad_precision }}
     steps:
     - name: Checkout
       uses: actions/checkout@v2
-    - name: Build container
-      run: docker build . -t fre-nctools
-    - name: Build and run tests
-      run: docker run -u root --entrypoint "/bin/sh" --env MPI --env QUAD_P fre-nctools /build.sh
+    - name: Configure
+      run: | 
+        autoreconf -i configure.ac
+        ./configure $MPI $QUAD_P
+    - name: Build tools
+      run: make
+    - name: Run tests
+      run: make -j check LOG_DRIVER_FLAGS=--comments
     - name: Save log file on failure
       uses: actions/upload-artifact@v2.2.1
       if: failure()
       with:
-          name: test-suite.log
-          path: t/test-suite.log
+        name: test-suite.log
+        path: t/test-suite.log