From 0f307c0449719b1c2edda6bce16aa5fa6e4a1bef Mon Sep 17 00:00:00 2001
From: Valentin Churavy <v.churavy@gmail.com>
Date: Wed, 5 Feb 2025 11:54:08 +0100
Subject: [PATCH] fixup! Always process code coverage

---
 .github/workflows/ci.yml | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 8333ec0aa5..f28d658e6b 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -127,11 +127,11 @@ jobs:
           PYTHON: ''
           TRIXI_TEST: ${{ matrix.trixi_test }}
       - uses: julia-actions/julia-processcoverage@v1
-        if: always()
+        if: ${{ !cancelled() }}
         with:
           directories: src,examples,ext
       - uses: codecov/codecov-action@v5
-        if: always()
+        if: ${{ !cancelled() }}
         with:
           files: ./lcov.info
           flags: unittests
@@ -155,7 +155,7 @@ jobs:
       # - Download and merge individual coverage reports in another step
       # - Upload only the merged coverage report to Coveralls
       - shell: bash
-        if: always()
+        if: ${{ !cancelled() }}
         run: |
           cp ./lcov.info ./lcov-${{ matrix.trixi_test }}-${{ matrix.os }}-${{ matrix.version }}-${{ matrix.arch }}.info
       - uses: actions/upload-artifact@v4
@@ -165,7 +165,7 @@ jobs:
 
   finish:
     needs: test
-    if: always()
+    if: ${{ !cancelled() }}
     runs-on: ubuntu-latest
     steps:
       # The standard setup of Coveralls is just annoying for parallel builds, see, e.g.,