Fixing slack notification

nextmv-io · Sep 23, 2024 · 0d7a140 · 0d7a140
1 parent 3b634f0
commit 0d7a140
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
@@ -227,7 +227,7 @@ jobs:
           export PATH=$PATH:~/.nextmv # Make CLI available in non-interactive shell
           export SLACK_NOTIFY_PARAM="" # Only notify slack for prod releases
           if [ ${{ matrix.environment }} == "prod" ]; then
-            export SLACK_NOTIFY_PARAM="--slack-url \"${{ secrets.SLACK_URL_MISSION_CONTROL }}\""
+            export SLACK_NOTIFY_PARAM="--slack-url ${{ secrets.SLACK_URL_MISSION_CONTROL }}"
           fi
           python main.py \
             --apps "${{ env.APPS }}" \

diff --git a/.nextmv/release/main.py b/.nextmv/release/main.py
@@ -86,7 +86,7 @@ def main():
             workflow_app=workflow_app,
         )
 
-        if options.slack_url is not None:
+        if options.slack_url:
             notify_slack(
                 url=options.slack_url,
                 app=name,