Merge pull request #1162 from zama-ai/ci/fix-macos-notif

fix(ci): always run notif job
zama-ai · Nov 27, 2024 · 2456b27 · 2456b27
2 parents 0958d62 + ff6fa72
commit 2456b27
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 1 deletion.
diff --git a/.github/workflows/concrete_compiler_test_macos_cpu.yml b/.github/workflows/concrete_compiler_test_macos_cpu.yml
@@ -96,6 +96,7 @@ jobs:
   slack-notif-macos:
     needs: ["build-and-test"]
     runs-on: "ubuntu-latest"
+    if: always()
     steps:
       - name: Slack Notification
         # we want to check that prepare_slack_notif was run

diff --git a/.github/workflows/concrete_python_release_cpu.yml b/.github/workflows/concrete_python_release_cpu.yml
@@ -450,6 +450,7 @@ jobs:
   slack-notif-macos:
     needs: ["test-macos"]
     runs-on: "ubuntu-latest"
+    if: always()
     steps:
       - name: Slack Notification
         # we want to check that prepare_slack_notif was run

diff --git a/.github/workflows/concrete_python_test_macos.yml b/.github/workflows/concrete_python_test_macos.yml
@@ -3,7 +3,7 @@ on:
   workflow_dispatch:
   pull_request:
     paths:
-      - .github/workflows/concrete_python_tests_macos.yml
+      - .github/workflows/concrete_python_test_macos.yml
   push:
     branches:
       - 'main'
@@ -122,6 +122,7 @@ jobs:
   slack-notif-macos:
     needs: ["concrete-python-test-pytest"]
     runs-on: "ubuntu-latest"
+    if: always()
     steps:
       - name: Slack Notification
         # we want to check that prepare_slack_notif was run