fix premsg

rffontenelle · rffontenelle · commit d7efe950f313 · 2021-11-04T10:54:13.000-03:00
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -6,11 +6,12 @@ on:
   push:
     paths:
       - '.github/workflows/build.yml'
+      - '.github/scripts/build.sh'
+      - '.github/scripts/prepmsg.sh'
       - '*.po'
       - '**/*.po'
 
 env:
-  # CPython branch where translations currently take place
   CPYTHON_BRANCH: '3.10'
   LANGUAGE: 'pt_BR'
 
@@ -56,9 +57,8 @@ jobs:
           path: cpython/Doc/build/html
       - name: Prepare error log for notification if it exists
         if: failure()
-        shell: bash
         run: |
-          sh scripts/setup-notif-message.sh error.log notify.log
+          sh scripts/prepmsg.sh error.log notify.log
         env:
           GITHUB_JOB: ${GITHUB_JOB}
           GITHUB_RUN_NUMBER: ${GITHUB_RUN_NUMBER}
diff --git a/scripts/prepmsg.sh b/scripts/prepmsg.sh
@@ -1,13 +1,14 @@
-#!/bin/sh
-# Prepare message for Telegram notification
-
-set -e
-Set -x
-
-[ $# -ne 2 ] && ( echo "Expected 1 input and 1 output files, got $#."; exit; )0
-[ ! -f $1 ]  && ( echo "Input file $1 not found, skipping."; exit; )
-
-URL="https://github.com/${GITHUB_REPOSITORY}/actions/runs/${GITHUB_RUN_NUMBER}"
-
-echo "❌ *${GITHUB_JOB}* ([ID&nbsp;*${GITHUB_RUN_NUMBER}*]($URL)):" > $2
-grep 'cpython/Doc/.*WARNING:' $1 | sed 's|.*/cpython/Doc|Doc|' | uniq >> $2
+#!/bin/sh
+# Prepare message for Telegram notification
+set -ex
+
+[ $# -ne 2 ] && ( echo "Expected 1 input and 1 output files, got $#"; exit; )
+[ ! -f $1 ]  && ( echo "Input file $1 not found, skipping."; exit; )
+for var in GITHUB_REPOSITORY GITHUB_RUN_NUMBER GITHUB_JOB; do
+  [ -z ${!var} ] && ( echo "${!var} is empty."; exit 1; )
+done
+
+URL="https://github.com/${GITHUB_REPOSITORY}/actions/runs/${GITHUB_RUN_NUMBER}"
+echo "❌ *${GITHUB_JOB}* ([ID&nbsp;*${GITHUB_RUN_NUMBER}*]($URL)):" > $2
+echo "" >> $2
+grep 'cpython/Doc/.*WARNING:' $1 | sed 's|.*/cpython/Doc|Doc|' | uniq >> $2
