Merge pull request #1125 from elementary-data/ele-1158-change-if-work…

…flow-syntax Fix condition syntax in workflow
elementary-data · Sep 3, 2023 · e637e83 · e637e83
2 parents 31ee080 + 39c11a7
commit e637e83
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/.github/workflows/test-all-warehouses.yml b/.github/workflows/test-all-warehouses.yml
@@ -53,8 +53,8 @@ jobs:
     if: |
       always() &&
       ! cancelled() &&
-      !contains(needs.test.result, 'success') &&
-      !contains(needs.test.result, 'cancelled')
+      ! contains(needs.test.result, 'success') &&
+      ! contains(needs.test.result, 'cancelled')
     uses: ./.github/workflows/notify_slack.yml
     with:
       result: "failure"

diff --git a/.github/workflows/test-github-action.yml b/.github/workflows/test-github-action.yml
@@ -107,8 +107,9 @@ jobs:
     needs: test
     if: |
       always() &&
-      contains(needs.test.result, 'failure') &&
       ! cancelled() &&
+      ! contains(needs.test.result, 'success') &&
+      ! contains(needs.test.result, 'cancelled')
       ${{ github.event_name == 'schedule' }}
     uses: ./.github/workflows/notify_slack.yml
     with: