Skip to content

Commit 193fb97

Browse files
committed
ci: do not report coverage on pull requests
The code coverage reports on pull requests has been inconsistent and confusing. Disable them for pull-requests. Signed-off-by: Anas Nashif <[email protected]>
1 parent 50c2a73 commit 193fb97

File tree

1 file changed

+33
-26
lines changed

1 file changed

+33
-26
lines changed

scripts/ci/run_ci.sh

Lines changed: 33 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -122,33 +122,40 @@ function handle_coverage() {
122122
echo "Calling gcovr"
123123
gcovr -r ${ZEPHYR_BASE} -x > shippable/codecoverage/coverage.xml;
124124

125-
# Capture data
126-
echo "Running lcov --capture ..."
127-
lcov --capture \
128-
--directory sanity-out/native_posix/ \
129-
--directory sanity-out/nrf52_bsim/ \
130-
--directory sanity-out/unit_testing/ \
131-
--directory bsim_bt_out/ \
132-
--output-file lcov.pre.info -q --rc lcov_branch_coverage=1;
133-
134-
# Remove noise
135-
echo "Exclude data from coverage report..."
136-
lcov -q \
137-
--remove lcov.pre.info mylib.c \
138-
--remove lcov.pre.info tests/\* \
139-
--remove lcov.pre.info samples/\* \
140-
--remove lcov.pre.info ext/\* \
141-
--remove lcov.pre.info *generated* \
142-
-o lcov.info --rc lcov_branch_coverage=1;
143-
144-
# Cleanup
145-
rm lcov.pre.info;
146-
rm -rf sanity-out out-2nd-pass;
147125

148-
# Upload to codecov.io
149-
echo "Upload coverage reports to codecov.io"
150-
bash <(curl -s https://codecov.io/bash) -f "lcov.info" -X coveragepy -X fixes;
151-
rm -f lcov.info;
126+
# Upload to codecov.io only on merged builds or if CODECOV_IO variable
127+
# is set.
128+
if [ -n "${CODECOV_IO}" -o -z "${PULL_REQUEST_NR}" ]; then
129+
# Capture data
130+
echo "Running lcov --capture ..."
131+
lcov --capture \
132+
--directory sanity-out/native_posix/ \
133+
--directory sanity-out/nrf52_bsim/ \
134+
--directory sanity-out/unit_testing/ \
135+
--directory bsim_bt_out/ \
136+
--output-file lcov.pre.info -q --rc lcov_branch_coverage=1;
137+
138+
# Remove noise
139+
echo "Exclude data from coverage report..."
140+
lcov -q \
141+
--remove lcov.pre.info mylib.c \
142+
--remove lcov.pre.info tests/\* \
143+
--remove lcov.pre.info samples/\* \
144+
--remove lcov.pre.info ext/\* \
145+
--remove lcov.pre.info *generated* \
146+
-o lcov.info --rc lcov_branch_coverage=1;
147+
148+
# Cleanup
149+
rm lcov.pre.info;
150+
rm -rf sanity-out out-2nd-pass;
151+
152+
# Upload to codecov.io
153+
echo "Upload coverage reports to codecov.io"
154+
bash <(curl -s https://codecov.io/bash) -f "lcov.info" -X coveragepy -X fixes;
155+
rm -f lcov.info;
156+
fi
157+
158+
rm -rf sanity-out out-2nd-pass;
152159

153160
}
154161

0 commit comments

Comments
 (0)