From 34f622c7bf6819f8876c4dc3c39c7842fcfa673c Mon Sep 17 00:00:00 2001
From: Youjung Kim <126618609+ykim-1@users.noreply.github.com>
Date: Mon, 7 Oct 2024 12:45:52 -0700
Subject: [PATCH] ci: Fix workflow failures in PRs due to slack notify step
 (#588)

* allow some vars to be shared

* add target branch

* remove target branch

* revert to push_request and only run slack notify when push to main
---
 .github/workflows/ci.yml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 83380bb35..dda016a45 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -26,7 +26,7 @@ jobs:
 
       - name: Fail if changes
         run: git diff-index --exit-code HEAD
-  
+
   test:
     runs-on: ubuntu-latest
     env:
@@ -81,7 +81,7 @@ jobs:
   notify-slack:
     runs-on: ubuntu-latest
     needs: [test]
-    if: always() && github.repository == 'linode/linodego' # Run even if integration tests fail and only on main repository
+    if: always() && github.ref == 'refs/heads/main' && github.event_name == 'push' # Run even if integration tests fail and only on main repository
 
     steps:
       - name: Notify Slack