Fix skipped CI reports (#2378)

* Add always() CI check to publish-qa-results * Add always() CI check to publish-tf-plan
usdigitalresponse · Dec 18, 2023 · 0ab2b3f · 0ab2b3f
1 parent a7236db
commit 0ab2b3f
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -19,7 +19,7 @@ jobs:
     permissions:
       contents: read
       pull-requests: write
-    if: needs.qa.result != 'skipped' || needs.qa.result != 'cancelled'
+    if: always() && (needs.qa.result != 'skipped' && needs.qa.result != 'cancelled')
     needs:
       - qa
     uses: "./.github/workflows/publish-qa-results.yml"
@@ -101,7 +101,7 @@ jobs:
     permissions:
       contents: read
       pull-requests: write
-    if: needs.tf-plan.result != 'skipped' || needs.tf-plan.result != 'cancelled'
+    if: always() && (needs.tf-plan.result != 'skipped' && needs.tf-plan.result != 'cancelled')
     needs:
       - tf-plan
     uses: ./.github/workflows/publish-terraform-plan.yml