[前][次][番号順一覧][スレッド一覧]

ruby-changes:74252

From: Takashi <ko1@a...>
Date: Wed, 26 Oct 2022 09:53:34 +0900 (JST)
Subject: [ruby-changes:74252] 1dd9511b67 (master): Use ruby/action-slack@v... and ref_name (#6633)

https://git.ruby-lang.org/ruby.git/commit/?id=1dd9511b67

From 1dd9511b6723b2781e65e2b4093a714576802302 Mon Sep 17 00:00:00 2001
From: Takashi Kokubun <takashikkbn@g...>
Date: Tue, 25 Oct 2022 17:53:08 -0700
Subject: Use ruby/action-slack@v... and ref_name (#6633)

It's moved from k0kubun to ruby org.

Also, we don't need JavaScript eval to generate branch if we use
github.ref_name, so v3.0.0 is a version that doesn't use eval.

Co-Authored-By: Nobuyoshi Nakada <nobu@r...>

Co-authored-by: Nobuyoshi Nakada <nobu@r...>
---
 .github/workflows/baseruby.yml           | 4 ++--
 .github/workflows/check_dependencies.yml | 4 ++--
 .github/workflows/check_misc.yml         | 4 ++--
 .github/workflows/compilers.yml          | 4 ++--
 .github/workflows/macos.yml              | 4 ++--
 .github/workflows/mingw.yml              | 4 ++--
 .github/workflows/mjit-bindgen.yml       | 4 ++--
 .github/workflows/mjit.yml               | 4 ++--
 .github/workflows/spec_guards.yml        | 4 ++--
 .github/workflows/ubuntu.yml             | 4 ++--
 .github/workflows/windows.yml            | 4 ++--
 .github/workflows/yjit-ubuntu.yml        | 4 ++--
 12 files changed, 24 insertions(+), 24 deletions(-)

diff --git a/.github/workflows/baseruby.yml b/.github/workflows/baseruby.yml
index 65d2813ac2..8152cad768 100644
--- a/.github/workflows/baseruby.yml
+++ b/.github/workflows/baseruby.yml
@@ -57,7 +57,7 @@ jobs: https://github.com/ruby/ruby/blob/trunk/.github/workflows/baseruby.yml#L57
       - run: make incs
       - run: make all
       - run: make test
-      - uses: k0kubun/action-slack@v...
+      - uses: ruby/action-slack@v...
         with:
           payload: |
             {
@@ -65,7 +65,7 @@ jobs: https://github.com/ruby/ruby/blob/trunk/.github/workflows/baseruby.yml#L65
               "env": "${{ github.workflow }} / BASERUBY @ ${{ matrix.ruby }}",
               "url": "https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }}",
               "commit": "${{ github.sha }}",
-              "branch": "${{ github.ref }}".split('/').reverse()[0]
+              "branch": "${{ github.ref_name }}"
             }
         env:
           SLACK_WEBHOOK_URL: ${{ secrets.SIMPLER_ALERTS_URL }} # ruby-lang slack: ruby/simpler-alerts-bot
diff --git a/.github/workflows/check_dependencies.yml b/.github/workflows/check_dependencies.yml
index 1753f46574..16e2829f73 100644
--- a/.github/workflows/check_dependencies.yml
+++ b/.github/workflows/check_dependencies.yml
@@ -56,7 +56,7 @@ jobs: https://github.com/ruby/ruby/blob/trunk/.github/workflows/check_dependencies.yml#L56
       - run: make all golf
       - run: ruby tool/update-deps --fix
       - run: git diff --no-ext-diff --ignore-submodules --exit-code
-      - uses: k0kubun/action-slack@v...
+      - uses: ruby/action-slack@v...
         with:
           payload: |
             {
@@ -64,7 +64,7 @@ jobs: https://github.com/ruby/ruby/blob/trunk/.github/workflows/check_dependencies.yml#L64
               "env": "${{ matrix.os }} / Dependencies need to update",
               "url": "https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }}",
               "commit": "${{ github.sha }}",
-              "branch": "${{ github.ref }}".split('/').reverse()[0]
+              "branch": "${{ github.ref_name }}"
             }
         env:
           SLACK_WEBHOOK_URL: ${{ secrets.SIMPLER_ALERTS_URL }} # ruby-lang slack: ruby/simpler-alerts-bot
diff --git a/.github/workflows/check_misc.yml b/.github/workflows/check_misc.yml
index 71ac956695..e897015d56 100644
--- a/.github/workflows/check_misc.yml
+++ b/.github/workflows/check_misc.yml
@@ -98,7 +98,7 @@ jobs: https://github.com/ruby/ruby/blob/trunk/.github/workflows/check_misc.yml#L98
           GIT_COMMITTER_NAME: git
         if: ${{ github.repository == 'ruby/ruby' && !startsWith(github.event_name, 'pull') && steps.diff.outcome == 'failure' }}
 
-      - uses: k0kubun/action-slack@v...
+      - uses: ruby/action-slack@v...
         with:
           payload: |
             {
@@ -106,7 +106,7 @@ jobs: https://github.com/ruby/ruby/blob/trunk/.github/workflows/check_misc.yml#L106
               "env": "${{ github.workflow }}",
               "url": "https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }}",
               "commit": "${{ github.sha }}",
-              "branch": "${{ github.ref }}".split('/').reverse()[0]
+              "branch": "${{ github.ref_name }}"
             }
         env:
           SLACK_WEBHOOK_URL: ${{ secrets.SIMPLER_ALERTS_URL }} # ruby-lang slack: ruby/simpler-alerts-bot
diff --git a/.github/workflows/compilers.yml b/.github/workflows/compilers.yml
index 8080963fa6..60fcc94c5e 100644
--- a/.github/workflows/compilers.yml
+++ b/.github/workflows/compilers.yml
@@ -268,7 +268,7 @@ jobs: https://github.com/ruby/ruby/blob/trunk/.github/workflows/compilers.yml#L268
       - run: make test-annocheck
         if: ${{ matrix.entry.check && endsWith(matrix.entry.name, 'annocheck') }}
 
-      - uses: k0kubun/action-slack@v...
+      - uses: ruby/action-slack@v...
         with:
           payload: |
             {
@@ -276,7 +276,7 @@ jobs: https://github.com/ruby/ruby/blob/trunk/.github/workflows/compilers.yml#L276
               "env": "${{ github.workflow }} / ${{ matrix.entry.name }}",
               "url": "https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }}",
               "commit": "${{ github.sha }}",
-              "branch": "${{ github.ref }}".split('/').reverse()[0]
+              "branch": "${{ github.ref_name }}"
             }
         env:
           SLACK_WEBHOOK_URL: ${{ secrets.SIMPLER_ALERTS_URL }} # ruby-lang slack: ruby/simpler-alerts-bot
diff --git a/.github/workflows/macos.yml b/.github/workflows/macos.yml
index 04446d3f05..e5f055f8c4 100644
--- a/.github/workflows/macos.yml
+++ b/.github/workflows/macos.yml
@@ -85,7 +85,7 @@ jobs: https://github.com/ruby/ruby/blob/trunk/.github/workflows/macos.yml#L85
           PRECHECK_BUNDLED_GEMS: "no"
         if: ${{ matrix.test_task == 'check' && matrix.skipped_tests != '' }}
         continue-on-error: ${{ matrix.continue-on-skipped_tests || false }}
-      - uses: k0kubun/action-slack@v...
+      - uses: ruby/action-slack@v...
         with:
           payload: |
             {
@@ -93,7 +93,7 @@ jobs: https://github.com/ruby/ruby/blob/trunk/.github/workflows/macos.yml#L93
               "env": "${{ matrix.os }} / ${{ matrix.test_task }}${{ matrix.configure }}",
               "url": "https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }}",
               "commit": "${{ github.sha }}",
-              "branch": "${{ github.ref }}".split('/').reverse()[0]
+              "branch": "${{ github.ref_name }}"
             }
         env:
           SLACK_WEBHOOK_URL: ${{ secrets.SIMPLER_ALERTS_URL }} # ruby-lang slack: ruby/simpler-alerts-bot
diff --git a/.github/workflows/mingw.yml b/.github/workflows/mingw.yml
index 5c8189428c..001fab9cf1 100644
--- a/.github/workflows/mingw.yml
+++ b/.github/workflows/mingw.yml
@@ -151,7 +151,7 @@ jobs: https://github.com/ruby/ruby/blob/trunk/.github/workflows/mingw.yml#L151
           make ${{ StartsWith(matrix.test_task, 'spec/') && matrix.test_task || 'test-spec' }}
         if: ${{matrix.test_task == 'check' || matrix.test_task == 'test-spec' || StartsWith(matrix.test_task, 'spec/')}}
 
-      - uses: k0kubun/action-slack@v...
+      - uses: ruby/action-slack@v...
         with:
           payload: |
             {
@@ -159,7 +159,7 @@ jobs: https://github.com/ruby/ruby/blob/trunk/.github/workflows/mingw.yml#L159
               "env": "${{ github.workflow }} ${{ matrix.msystem }} / ${{ matrix.test_task }}",
               "url": "https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }}",
               "commit": "${{ github.sha }}",
-              "branch": "${{ github.ref }}".split('/').reverse()[0]
+              "branch": "${{ github.ref_name }}"
             }
         env:
           SLACK_WEBHOOK_URL: ${{ secrets.SIMPLER_ALERTS_URL }} # ruby-lang slack: ruby/simpler-alerts-bot
diff --git a/.github/workflows/mjit-bindgen.yml b/.github/workflows/mjit-bindgen.yml
index e1d22d91f2..1252d3b509 100644
--- a/.github/workflows/mjit-bindgen.yml
+++ b/.github/workflows/mjit-bindgen.yml
@@ -83,7 +83,7 @@ jobs: https://github.com/ruby/ruby/blob/trunk/.github/workflows/mjit-bindgen.yml#L83
       - run: $SETARCH make ${{ matrix.task }}
       - run: git diff --exit-code
         working-directory: src
-      - uses: k0kubun/action-slack@v...
+      - uses: ruby/action-slack@v...
         with:
           payload: |
             {
@@ -91,7 +91,7 @@ jobs: https://github.com/ruby/ruby/blob/trunk/.github/workflows/mjit-bindgen.yml#L91
               "env": "${{ matrix.os }} / ${{ matrix.test_task }}${{ matrix.configure }}",
               "url": "https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }}",
               "commit": "${{ github.sha }}",
-              "branch": "${{ github.ref }}".split('/').reverse()[0]
+              "branch": "${{ github.ref_name }}"
             }
         env:
           SLACK_WEBHOOK_URL: ${{ secrets.SIMPLER_ALERTS_URL }} # ruby-lang slack: ruby/simpler-alerts-bot
diff --git a/.github/workflows/mjit.yml b/.github/workflows/mjit.yml
index 89bc0a226b..c468e01811 100644
--- a/.github/workflows/mjit.yml
+++ b/.github/workflows/mjit.yml
@@ -84,7 +84,7 @@ jobs: https://github.com/ruby/ruby/blob/trunk/.github/workflows/mjit.yml#L84
           ulimit -c unlimited
           make -s test-spec RUN_OPTS="$RUN_OPTS"
         timeout-minutes: 60
-      - uses: k0kubun/action-slack@v...
+      - uses: ruby/action-slack@v...
         with:
           payload: |
             {
@@ -92,7 +92,7 @@ jobs: https://github.com/ruby/ruby/blob/trunk/.github/workflows/mjit.yml#L92
               "env": "${{ github.workflow }} / ${{ matrix.test_task }} ${{ matrix.jit_opts }}",
           (... truncated)

--
ML: ruby-changes@q...
Info: http://www.atdot.net/~ko1/quickml/

[前][次][番号順一覧][スレッド一覧]