Add result CI job to enable auto-merge (#12)

* Add `result` CI job to enable auto-merge

* Avoid triggering duplicated jobs on PRs

* Drop cancel-in-progress

We don't run as many jobs as ruby/ruby CI. We shouldn't need this here.

* Fix a typo

* Drop the concurrency group too

Similarly, we shouldn't need this in the Shopify/zjit repo

Author: k0kubun

.github/workflows/yjit-macos.yml

@@ -1,6 +1,8 @@
 name: YJIT macOS Arm64
 on:
   push:
+    branches:
+      - master
     paths-ignore:
       - 'doc/**'
       - '**/man/*'
@@ -9,14 +11,14 @@ on:
       - '**/.document'
       - '.*.yml'
   pull_request:
+    types:
+      - opened
+      - synchronize
+      - reopened
     # Do not use paths-ignore for required status checks
     # https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/collaborating-on-repositories-with-code-quality-features/troubleshooting-required-status-checks#handling-skipped-but-required-checks
   merge_group:
 
-concurrency:
-  group: ${{ github.workflow }} / ${{ startsWith(github.event_name, 'pull') && github.ref_name || github.sha }}
-  cancel-in-progress: ${{ startsWith(github.event_name, 'pull') }}
-
 permissions:
   contents: read
 
@@ -162,7 +164,8 @@ jobs:
     runs-on: macos-14
     needs: [make]
     steps:
-      - run: exit 1
+      - name: ${{ github.workflow }} jobs have failed
+        run: exit 1
         working-directory:
         if: ${{ contains(needs.*.result, 'failure') || contains(needs.*.result, 'cancelled') }}
 

.github/workflows/zjit-macos.yml

@@ -102,11 +102,15 @@ jobs:
           BTESTS: ${{ matrix.btests }}
         continue-on-error: ${{ matrix.continue-on-test_task || false }}
 
-      - uses: ./.github/actions/slack
-        with:
-          label: ${{ matrix.test_task }} ${{ matrix.configure }} ${{ matrix.zjit_opts }}
-          SLACK_WEBHOOK_URL: ${{ secrets.SIMPLER_ALERTS_URL }} # ruby-lang slack: ruby/simpler-alerts-bot
-        if: ${{ failure() }}
+  result:
+    if: ${{ always() }}
+    name: ${{ github.workflow }} result
+    runs-on: macos-14
+    needs: [make]
+    steps:
+      - run: exit 1
+        working-directory:
+        if: ${{ contains(needs.*.result, 'failure') || contains(needs.*.result, 'cancelled') }}
 
 defaults:
   run:

.github/workflows/zjit-ubuntu.yml

@@ -1,6 +1,8 @@
 name: ZJIT Ubuntu
 on:
   push:
+    branches:
+      - master
     paths-ignore:
       - 'doc/**'
       - '**/man/*'
@@ -9,14 +11,14 @@ on:
       - '**/.document'
       - '.*.yml'
   pull_request:
+    types:
+      - opened
+      - synchronize
+      - reopened
     # Do not use paths-ignore for required status checks
     # https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/collaborating-on-repositories-with-code-quality-features/troubleshooting-required-status-checks#handling-skipped-but-required-checks
   merge_group:
 
-concurrency:
-  group: ${{ github.workflow }} / ${{ startsWith(github.event_name, 'pull') && github.ref_name || github.sha }}
-  cancel-in-progress: ${{ startsWith(github.event_name, 'pull') }}
-
 permissions:
   contents: read
 
@@ -128,11 +130,16 @@ jobs:
           BTESTS: ${{ matrix.btests }}
         continue-on-error: ${{ matrix.continue-on-test_task || false }}
 
-      - uses: ./.github/actions/slack
-        with:
-          label: ${{ matrix.test_task }} ${{ matrix.configure }}
-          SLACK_WEBHOOK_URL: ${{ secrets.SIMPLER_ALERTS_URL }} # ruby-lang slack: ruby/simpler-alerts-bot
-        if: ${{ failure() }}
+  result:
+    if: ${{ always() }}
+    name: ${{ github.workflow }} result
+    runs-on: ubuntu-22.04
+    needs: [make]
+    steps:
+      - name: ${{ github.workflow }} jobs have failed
+        run: exit 1
+        working-directory:
+        if: ${{ contains(needs.*.result, 'failure') || contains(needs.*.result, 'cancelled') }}
 
 defaults:
   run: