From 7659a8eb37c8e09eecf9a5df79661df79a105409 Mon Sep 17 00:00:00 2001 From: Stas Bekman Date: Tue, 18 Aug 2020 07:21:13 -0700 Subject: [PATCH] fix incorrect codecov reports (#6553) As discussed at https://github.com/huggingface/transformers/issues/6317 codecov currently sends an invalid report when it fails to find a code coverage report for the base it checks against, so this gets fixed by: - require_base: yes # don't report if there is no base coverage report let's add this for clarity, this supposedly is already the default. - require_head: yes # don't report if there is no head coverage report and perhaps no point reporting on doc changes as they don't make any difference and it just generates noise: - require_changes: true # only comment if there was change in coverage --- codecov.yml | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/codecov.yml b/codecov.yml index bf6f43d9c3..ecacb3725f 100644 --- a/codecov.yml +++ b/codecov.yml @@ -4,3 +4,7 @@ coverage: default: informational: true patch: off +comment: + require_changes: true # only comment if there was change in coverage + require_head: yes # don't report if there is no head coverage report + require_base: yes # don't report if there is no base coverage report