Skip to content

Commit afca13e

Browse files
committed
Use Valgrind instead of AddressSanitizer to avoid the need for instrumentation and extend coverage to doctests.
1 parent aaf890b commit afca13e

File tree

1 file changed

+5
-6
lines changed

1 file changed

+5
-6
lines changed

.github/workflows/ci.yml

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -97,25 +97,24 @@ jobs:
9797
manylinux: auto
9898
args: --manifest-path examples/simple/Cargo.toml
9999

100-
address-sanitizer:
100+
valgrind:
101101
runs-on: ubuntu-22.04
102102
needs: [lint, check-msrv, examples]
103103
steps:
104104
- uses: actions/checkout@v3
105105
- uses: actions-rs/toolchain@v1
106106
with:
107-
toolchain: nightly
107+
toolchain: stable
108108
profile: minimal
109-
components: rust-src
110109
default: true
111110
- uses: Swatinem/rust-cache@v2
112111
continue-on-error: true
113112
- run: |
113+
sudo apt-get install --yes valgrind
114114
pip install numpy
115-
cargo test -Zbuild-std --target x86_64-unknown-linux-gnu --release --lib --tests
115+
cargo test --all-features --release
116116
env:
117-
RUSTFLAGS: -Zsanitizer=address
118-
ASAN_OPTIONS: detect_leaks=0
117+
CARGO_TARGET_X86_64_UNKNOWN_LINUX_GNU_RUNNER: valgrind --leak-check=no --error-exitcode=1
119118
120119
check-msrv:
121120
runs-on: ubuntu-latest

0 commit comments

Comments
 (0)