Merge pull request #13517 from MoonlightSentinel/azure-fail

Coverage: Don't fail if reports could not be uploaded to CodeCov Signed-off-by: Nicholas Wilson <thewilsonator@users.noreply.github.com> Merged-on-behalf-of: Nicholas Wilson <thewilsonator@users.noreply.github.com>
dlang · Jan 13, 2022 · 14b9c01 · 14b9c01
2 parents afa5ae0 + bb9fec3
commit 14b9c01
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/.azure-pipelines/windows.sh b/.azure-pipelines/windows.sh
@@ -133,9 +133,6 @@ CC="$CC" ./run --environment --jobs=$N "${targets[@]}" "${args[@]}"
 ################################################################################
 
 if [ "${DMD_TEST_COVERAGE:-0}" = "1" ] ; then
-    cd $DMD_DIR
-    OS_NAME=windows source ci/codecov.sh
-
     # Skip druntime & phobos tests
     exit 0
 fi

diff --git a/.azure-pipelines/windows.yml b/.azure-pipelines/windows.yml
@@ -12,3 +12,11 @@ steps:
       bash --version
       sh --login .azure-pipelines/windows.sh
     displayName: Build and test
+
+  - bash: |
+      set -eux
+      source .azure-pipelines/lib.sh
+      OS_NAME=windows source ci/codecov.sh
+    displayName: "Upload coverage report"
+    condition: eq( variables.DMD_TEST_COVERAGE, 1)
+    continueOnError: true