Skip to content

Commit 4ff2a50

Browse files
kim-emclaude
andcommitted
fix: webhook --fail flag and zulip validation always()
- Add --fail to curl so HTTP errors are not silently ignored - Add always() to Zulip validation so it runs even when nanoda fails Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent 7d0388a commit 4ff2a50

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

.github/workflows/nanoda-daily.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ jobs:
8787
EMOJI=$([[ "$STATUS" == "success" ]] && echo "✅" || echo "❌")
8888
RUN_URL="${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}"
8989
90-
curl -X POST "$WEBHOOK_URL" \
90+
curl --fail -X POST "$WEBHOOK_URL" \
9191
-H "Content-Type: application/json" \
9292
-d "{
9393
\"text\": \"$EMOJI nanoda daily verification: $STATUS\",
@@ -104,7 +104,7 @@ jobs:
104104
shell: bash
105105

106106
- name: Validate Zulip configuration
107-
if: inputs.notify == 'zulip'
107+
if: always() && inputs.notify == 'zulip'
108108
env:
109109
ZULIP_ORG_URL: ${{ inputs.zulip-org-url }}
110110
ZULIP_API_KEY: ${{ secrets.zulip-api-key }}

0 commit comments

Comments
 (0)