Changeset 51952

Timestamp:

10/29/2021 07:40:26 PM (3 years ago)

Author:

desrosj

Message:

Build/Test Tools: Escape $ within commit messages for `$variables.

This ensures the variables are preserved in the Slack message.

Props ocean90, desrosj.
See #53363.

File:

• trunk/.github/workflows/slack-notifications.yml (modified) (1 diff)

trunk/.github/workflows/slack-notifications.yml

@@ -111 +111 @@
         id: commit-message
         run: |
-          COMMIT_MESSAGE=$(cat <<'EOF' | awk 'NR==1' | sed 's/`/\\`/g' | sed 's/\"/\\\\"/g'
+          COMMIT_MESSAGE=$(cat <<'EOF' | awk 'NR==1' | sed 's/`/\\`/g' | sed 's/\"/\\\\"/g' | sed 's/\$/\\$/g'
           ${{ github.event_name == 'workflow_run' && github.event.workflow_run.head_commit.message || github.event.head_commit.message }}
           EOF