From fd20b32130f52f4a7799a8d7a367f5b39636bc6a Mon Sep 17 00:00:00 2001
From: Takashi Kokubun <takashikkbn@gmail.com>
Date: Thu, 22 Aug 2019 23:10:54 +0900
Subject: Make GitHub Actions Slack notification consistent

with Travis, rather than AppVeyor.

Formerly it was made similar to AppVeyor to provide some normal set of
CI failure notification. But for some reason people preferred a shorter
variant and introduced a1d606c079f6c3d1779d885e0bf2e3991251609e and
d8d8015b93c6daa8d8433895464db3493a2056e2.

Instead of AppVeyor format, this commit chose Travis-like format to achieve
consistency and to include usual CI-failure information, while keeping
it one-liner for people who prefer short notifications.

Note that this shrinks the 40-char sha to 10-char, using the new feature
of k0kubun/action-slack@v2.0.0:
https://github.com/k0kubun/action-slack/commit/1c88a05dac664cbafa1c99a37f292ed23ac1c289
---
 .github/workflows/macos.yml   | 6 ++++--
 .github/workflows/ubuntu.yml  | 6 ++++--
 .github/workflows/windows.yml | 6 ++++--
 3 files changed, 12 insertions(+), 6 deletions(-)

(limited to '.github')

diff --git a/.github/workflows/macos.yml b/.github/workflows/macos.yml
index 3272dc1561..2e53d52f59 100644
--- a/.github/workflows/macos.yml
+++ b/.github/workflows/macos.yml
@@ -57,12 +57,14 @@ jobs:
         if: matrix.test_task == 'test-bundled-gems' && !contains(github.event.head_commit.message, '[ci skip]')
       - name: Leaked Globals
         run: make -s leaked-globals
-      - uses: k0kubun/action-slack@v1.0.0
+      - uses: k0kubun/action-slack@v2.0.0
         with:
           payload: |
             {
               "attachments": [{
-                "text": "<https://github.com/${{ github.repository }}/commit/${{ github.sha }}/checks|${{ job.status }}: ${{ github.workflow }} / ${{ matrix.test_task }}> (<https://github.com/${{ github.repository }}/commit/${{ github.sha }}|${{ github.sha }}>)",
+                "text": "${{ job.status}}: <https://github.com/${{ github.repository }}/commit/${{ github.sha }}/checks|${{ github.workflow }} / ${{ matrix.test_task }}> " +
+                        "(<https://github.com/${{ github.repository }}/commit/${{ github.sha }}|" + "${{ github.sha }}".substring(0, 10) + ">) " +
+                        "of ${{ github.repository }}@" + "${{ github.ref }}".split('/').reverse()[0] + " by ${{ github.event.head_commit.committer.name }}",
                 "color": "danger"
               }]
             }
diff --git a/.github/workflows/ubuntu.yml b/.github/workflows/ubuntu.yml
index b3b5c70526..1c82c576c5 100644
--- a/.github/workflows/ubuntu.yml
+++ b/.github/workflows/ubuntu.yml
@@ -65,12 +65,14 @@ jobs:
         env:
           GITHUB_CONTEXT: ${{ toJson(github) }}
         if: failure() && github.event_name == 'push'
-      - uses: k0kubun/action-slack@v1.0.0
+      - uses: k0kubun/action-slack@v2.0.0
         with:
           payload: |
             {
               "attachments": [{
-                "text": "<https://github.com/${{ github.repository }}/commit/${{ github.sha }}/checks|${{ job.status }}: ${{ github.workflow }} / ${{ matrix.test_task }}> (<https://github.com/${{ github.repository }}/commit/${{ github.sha }}|${{ github.sha }}>)",
+                "text": "${{ job.status}}: <https://github.com/${{ github.repository }}/commit/${{ github.sha }}/checks|${{ github.workflow }} / ${{ matrix.test_task }}> " +
+                        "(<https://github.com/${{ github.repository }}/commit/${{ github.sha }}|" + "${{ github.sha }}".substring(0, 10) + ">) " +
+                        "of ${{ github.repository }}@" + "${{ github.ref }}".split('/').reverse()[0] + " by ${{ github.event.head_commit.committer.name }}",
                 "color": "danger"
               }]
             }
diff --git a/.github/workflows/windows.yml b/.github/workflows/windows.yml
index 2f3069e436..3987153ae3 100644
--- a/.github/workflows/windows.yml
+++ b/.github/workflows/windows.yml
@@ -54,12 +54,14 @@ jobs:
           call "C:\Program Files (x86)\Microsoft Visual Studio\${{ matrix.vs }}\Enterprise\VC\Auxiliary\Build\vcvars64.bat"
           nmake ${{ matrix.test_task }}
         if: "!contains(github.event.head_commit.message, '[ci skip]')"
-      - uses: k0kubun/action-slack@v1.0.0
+      - uses: k0kubun/action-slack@v2.0.0
         with:
           payload: |
             {
               "attachments": [{
-                "text": "<https://github.com/${{ github.repository }}/commit/${{ github.sha }}/checks|${{ job.status }}: ${{ github.workflow }} / ${{ matrix.test_task }}> (<https://github.com/${{ github.repository }}/commit/${{ github.sha }}|${{ github.sha }}>)",
+                "text": "${{ job.status}}: <https://github.com/${{ github.repository }}/commit/${{ github.sha }}/checks|${{ github.workflow }} / ${{ matrix.test_task }}> " +
+                        "(<https://github.com/${{ github.repository }}/commit/${{ github.sha }}|" + "${{ github.sha }}".substring(0, 10) + ">) " +
+                        "of ${{ github.repository }}@" + "${{ github.ref }}".split('/').reverse()[0] + " by ${{ github.event.head_commit.committer.name }}",
                 "color": "danger"
               }]
             }
-- 
cgit v1.2.3