Merge pull request #2209 from fluentmigrator/copilot/update-upload-artifact-handling

jzabroski · web-flow · commit d4918cf3ef10 · 2025-12-04T18:40:49.000-05:00
Add if condition to Upload Test Result Files step to handle test failures
diff --git a/.github/workflows/pull-requests.yaml b/.github/workflows/pull-requests.yaml
@@ -127,6 +127,7 @@ jobs:
 
     - name: Combine Coverage Reports
       uses: danielpalme/ReportGenerator-GitHub-Action@v5.5.0
+      if: '!cancelled()'
       with:
         reports: "**/*.cobertura.xml"
         targetdir: "${{ github.workspace }}"
@@ -138,6 +139,7 @@ jobs:
 
     - name: Upload Combined Coverage XML
       uses: actions/upload-artifact@v4
+      if: '!cancelled()'
       with:
         name: coverage-${{ matrix.os }}-${{ matrix.dotnet-tfm }}
         path: ${{ github.workspace }}/Cobertura.xml
@@ -166,6 +168,7 @@ jobs:
 
     - name: Upload Test Result Files
       uses: actions/upload-artifact@v4
+      if: '!cancelled()'
       with:
         name: test-results-${{ matrix.os }}-${{ matrix.dotnet-tfm }}
         path: ${{ github.workspace }}/**/TestResults-${{ matrix.dotnet-tfm }}/**/*
