From 3f2467b6bc68a1a1112125d83cf449026b141dde Mon Sep 17 00:00:00 2001 From: Greg Sleap Date: Mon, 17 Jan 2022 14:42:08 +0800 Subject: [PATCH] Implemented fix for coverage so it does not label assert lines as partially covered. See: https://github.com/mozilla/grcov/issues/476 and https://github.com/MWATelescope/Marlu/commit/bc34f29651ef3502629b053e627f052fed86fba6 for more info. Should restore my coverage back to normal levels, I hope --- .github/workflows/coverage.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/coverage.yml b/.github/workflows/coverage.yml index 512ea6e..db1ce7a 100644 --- a/.github/workflows/coverage.yml +++ b/.github/workflows/coverage.yml @@ -59,7 +59,7 @@ jobs: cargo build cargo test zip -0 ccov.zip `find . \( -name "mwalib*.gc*" \) -print` - grcov ccov.zip -s . -t lcov --llvm --branch --ignore-not-existing --ignore "/*" -o coverage/coverage.lcov + grcov ccov.zip -s . -t lcov --llvm --branch --ignore-not-existing --ignore "/*" --excl-br-line "^.*((debug_)?assert(_eq|_ne|_abs_diff_(eq|ne))?!|#\[derive\()" -o coverage/coverage.lcov - name: Upload reports to codecov.io uses: codecov/codecov-action@v1