ci: refactor release flow for bindings (#741)

Author: everpcpc

.github/workflows/bindings.nodejs.yml

@@ -1,24 +1,13 @@
 name: Bindings Node.js
 
 on:
-  push:
-    branches:
-      - main
-    tags:
-      - v*
-  pull_request:
-    branches:
-      - main
-    paths:
-      - "core/**"
-      - "sql/**"
-      - "driver/**"
-      - "bindings/nodejs/**"
-      - ".github/workflows/bindings.nodejs.yml"
-
-concurrency:
-  group: ${{ github.workflow }}-${{ github.ref }}-${{ github.event_name }}
-  cancel-in-progress: true
+  workflow_call:
+    inputs:
+      publish:
+        description: 'Whether to publish to npm'
+        type: boolean
+        required: false
+        default: false
 
 jobs:
   check:
@@ -131,7 +120,7 @@ jobs:
       - run: make -C tests test-bindings-nodejs
 
   publish:
-    if: startsWith(github.ref, 'refs/tags/v')
+    if: inputs.publish == true
     needs: [build, integration]
     runs-on: ubuntu-latest
     permissions:

.github/workflows/bindings.python.yml

@@ -1,24 +1,13 @@
 name: Bindings Python
 
 on:
-  push:
-    branches:
-      - main
-    tags:
-      - v*
-  pull_request:
-    branches:
-      - main
-    paths:
-      - "core/**"
-      - "sql/**"
-      - "driver/**"
-      - "bindings/python/**"
-      - ".github/workflows/bindings.python.yml"
-
-concurrency:
-  group: ${{ github.workflow }}-${{ github.ref }}-${{ github.event_name }}
-  cancel-in-progress: true
+  workflow_call:
+    inputs:
+      publish:
+        description: 'Whether to publish to PyPI'
+        type: boolean
+        required: false
+        default: false
 
 jobs:
   check:
@@ -208,7 +197,7 @@ jobs:
 
 
   publish:
-    if: startsWith(github.ref, 'refs/tags/v')
+    if: inputs.publish == true
     needs: [check, build, integration]
     runs-on: ubuntu-latest
     permissions:

.github/workflows/ci.yml

@@ -8,7 +8,7 @@ on:
 
 concurrency:
   group: ${{ github.workflow }}-${{ github.ref }}-${{ github.event_name }}
-  cancel-in-progress: true
+  cancel-in-progress: ${{ github.event_name == 'pull_request' }}
 
 jobs:
   check:
@@ -89,3 +89,17 @@ jobs:
       - run: make -C tests test-core
       - run: make -C tests test-driver
       - run: make -C tests test-bendsql
+
+  bindings_python:
+    needs: check
+    uses: ./.github/workflows/bindings.python.yml
+    with:
+      publish: false
+    secrets: inherit
+
+  bindings_nodejs:
+    needs: check
+    uses: ./.github/workflows/bindings.nodejs.yml
+    with:
+      publish: false
+    secrets: inherit

.github/workflows/release.yml

@@ -244,3 +244,19 @@ jobs:
           GH_TOKEN: ${{ github.token }}
         run: |
           gh release upload ${{ needs.check.outputs.new_version }} dist/pkg/* --clobber
+
+  bindings_python:
+    needs: [check, build_linux]
+    if: needs.check.outputs.version_changed == 'true'
+    uses: ./.github/workflows/bindings.python.yml
+    with:
+      publish: true
+    secrets: inherit
+
+  bindings_nodejs:
+    needs: [check, build_linux]
+    if: needs.check.outputs.version_changed == 'true'
+    uses: ./.github/workflows/bindings.nodejs.yml
+    with:
+      publish: true
+    secrets: inherit