Skip to content

Commit ae0525d

Browse files
committed
Add valgrind job
1 parent 7cffb77 commit ae0525d

1 file changed

Lines changed: 3 additions & 19 deletions

File tree

.github/workflows/c-cpp.yml

Lines changed: 3 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ jobs:
2323
run: |
2424
sudo apt-get update
2525
sudo apt-get install -y build-essential cmake lcov
26-
sudo apt-get install -y libgtest-dev googletest
26+
sudo apt-get install -y libgtest-dev googletest valgrind
2727
2828
- name: Configure CMake
2929
run: |
@@ -73,26 +73,10 @@ jobs:
7373
file: coverage.info
7474
fail_ci_if_error: false
7575

76-
valgrind:
77-
needs: build-and-test
78-
runs-on: ubuntu-latest
79-
steps:
80-
- uses: actions/checkout@v4
81-
82-
- name: Install Valgrind
83-
run: |
84-
sudo apt-get update
85-
sudo apt-get install -y build-essential cmake lcov
86-
sudo apt-get install -y libgtest-dev googletest valgrind
87-
88-
- name: Build Debug version
89-
run: |
90-
cmake -B build-debug -DCMAKE_BUILD_TYPE=Debug
91-
cmake --build build-debug --target ini_parser_tests
92-
9376
- name: Valgrind memory check
77+
working-directory: ./build
9478
run: |
9579
valgrind --leak-check=full \
9680
--track-origins=yes \
9781
--error-exitcode=1 \
98-
./build-debug/ini_parser_tests
82+
./ini_parser_tests

0 commit comments

Comments
 (0)