Skip to content

Commit ebecdcf

Browse files
aaronj0vgvassilev
authored andcommitted
[ci] No longer do true for cppyy valgrind
- Part of utilizing suppressions
1 parent d14136e commit ebecdcf

File tree

1 file changed

+2
-7
lines changed

1 file changed

+2
-7
lines changed

.github/workflows/ci.yml

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1025,13 +1025,8 @@ jobs:
10251025
if [[ "${{ matrix.os }}" == macos-* ]]; then
10261026
echo "Skipping Valgrind checks on OS X"
10271027
else
1028-
if [[ "${{ matrix.clang-runtime }}" == "17" ]]; then
1029-
echo "Valgrind reports true for clang-runtime 17 due to problems with LLVM"
1030-
valgrind --show-error-list=yes --error-exitcode=1 --track-origins=yes --suppressions=../etc/valgrind-cppyy-cling.supp python -m pytest -m "not xfail" -v || true
1031-
else
1032-
echo "Running valgrind on passing tests"
1033-
valgrind --show-error-list=yes --error-exitcode=1 --track-origins=yes --suppressions=../etc/valgrind-cppyy-cling.supp python -m pytest -m "not xfail" -v
1034-
fi
1028+
echo "Running valgrind on passing tests"
1029+
valgrind --show-error-list=yes --error-exitcode=1 --track-origins=yes --suppressions=../etc/valgrind-cppyy-cling.supp python -m pytest -m "not xfail" -v
10351030
fi
10361031
export RETCODE=+$?
10371032
echo ::endgroup::

0 commit comments

Comments
 (0)