From 4b3d19b238d6b82e88256f5da79713ca7a77732a Mon Sep 17 00:00:00 2001 From: Arpit Mohan Date: Thu, 18 Jan 2024 12:45:23 +0530 Subject: [PATCH] ci: Fixing Slack notification in test-build-docker-image (#30407) Fixing minor syntax issues when notifying on Slack on failure of the Test build Docker workflow. ## Summary by CodeRabbit - **Chores** - Improved Slack notification messages by adjusting link formatting. --- .github/workflows/test-build-docker-image.yml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/.github/workflows/test-build-docker-image.yml b/.github/workflows/test-build-docker-image.yml index c93b10b3d8..0ca0c30b85 100644 --- a/.github/workflows/test-build-docker-image.yml +++ b/.github/workflows/test-build-docker-image.yml @@ -403,10 +403,10 @@ jobs: merged_upto_sha='${{ steps.merge.outputs.merged_upto_sha }}' merged_upto_sha_short="$(echo "$merged_upto_sha" | grep -o '^.\{8\}' || true)" - details="🚨 TBP workflow failed in <$run_url|${{ github.run_id }}/attempts/${{ github.run_attempt }}>. + details="🚨 TBP workflow failed in <$run_url|${{ github.run_id }}/attempts/${{ github.run_attempt }}>." - # This unweildy horror of a sed command, converts standard Markdown links to Slack's unweildy link syntax. - slack_message="$(echo "$details" | sed -E 's/\[([^]]+)\]\(([^)]+)\)/<\2|\1>/g')" + # This unwieldy horror of a sed command, converts standard Markdown links to Slack's unwieldy link syntax. + slack_message=$(echo "$details" | sed -E 's/\[([^]]+)\]\(([^)]+)\)/<\2|\1>/g') # This is the ChannelId of the tech channel. body="$(jq -nc \