Limit testing with valgrind

valgrind output is very verbose and it's hard to see the failure.
pkubowicz · Dec 17, 2023 · 13ecf24 · 13ecf24
1 parent d43f893
commit 13ecf24
Showing 1 changed file with 18 additions and 3 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -10,22 +10,37 @@ jobs:
         include:
         - os: ubuntu-latest
           compiler: gcc
+          valgrind: true
         - os: ubuntu-latest
           compiler: clang
+          valgrind: true
         - os: ubuntu-20.04
           compiler: gcc
+          valgrind: false
         - os: ubuntu-20.04
           compiler: clang
+          valgrind: false
 
     runs-on: ${{ matrix.os }}
 
     steps:
       - uses: actions/checkout@v4
       - name: Setup dependencies
-        run: sudo apt install flex valgrind libfl-dev
-      - run: |
+        run: sudo apt install libfl-dev
+      - name: Compile
+        run: |
           export CC=${{matrix.compiler}}
           $CC --version
           make
+      - name: Test with valgrind
+        if: ${{ matrix.valgrind == true }}
+        run: |
+          sudo apt install valgrind
           ./test.pl --valgrind
-          make package
+      - name: Test without valgrind
+        if: ${{ matrix.valgrind != true }}
+        run: |
+          ./test.pl
+      - name: Test packaging
+        if: ${{ matrix.valgrind != true }}
+        run: make package