Skip to content

Commit bfd7ddb

Browse files
committed
CI: Print valgrind memcheck log on success or failure
Move the memcheck log printing to a separate step that runs with always() condition, so the log is printed regardless of whether the valgrind check passes or fails. This helps debug valgrind failures in CI.
1 parent d6cda56 commit bfd7ddb

File tree

1 file changed

+9
-1
lines changed

1 file changed

+9
-1
lines changed

.github/workflows/verify-build.yml

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -624,9 +624,17 @@ jobs:
624624
run: |
625625
cd build ;
626626
make check-valgrind ;
627-
cat test/test-suite-memcheck.log ;
628627
if: ${{ matrix.build-type == 'valgrind' }}
629628

629+
- name: Print Valgrind memcheck results
630+
shell: bash
631+
run: |
632+
cd build ;
633+
if [ -f test/test-suite-memcheck.log ]; then
634+
cat test/test-suite-memcheck.log ;
635+
fi
636+
if: ${{ always() && matrix.build-type == 'valgrind' }}
637+
630638
- name: Run cppcheck
631639
run: |
632640
cd src/ ;

0 commit comments

Comments
 (0)