From ae89c363910916f3e073ce790372a66c670f6828 Mon Sep 17 00:00:00 2001
From: David Li <li.davidm96@gmail.com>
Date: Tue, 27 Jan 2026 11:29:15 +0900
Subject: [PATCH] feat: change concurrency groups

---
 adbc_drivers_dev/templates/test.yaml | 23 +++++++++++++++--------
 adbc_drivers_dev/workflow.py         |  1 -
 2 files changed, 15 insertions(+), 9 deletions(-)

diff --git a/adbc_drivers_dev/templates/test.yaml b/adbc_drivers_dev/templates/test.yaml
index 8024c1f..99d342b 100644
--- a/adbc_drivers_dev/templates/test.yaml
+++ b/adbc_drivers_dev/templates/test.yaml
@@ -29,17 +29,16 @@
 name: <{lang_human}> <{workflow_name}>
 
 on:
+<% if not release %>
   pull_request:
     branches:
       - main
     paths:
-<% if not release %>
       - "<{lang_subdir}>/**"
-<% endif %>
 <% for path in pull_request_trigger_paths %>
       - <{path}>
 <% endfor %>
-
+<% endif %>
   push:
 <% if release %>
     tags:
@@ -76,8 +75,12 @@ on:
 <% endif %>
 
 concurrency:
+<% if environment %>
   # Must share concurrency group with release workflow since it also builds/tests
-  group: ${{ github.repository }}-${{ github.ref }}-ci
+  group: ${{ github.repository }}-${{ github.ref }}-<{environment}>
+<% else %>
+  group: ${{ github.repository }}-${{ github.ref }}-${{ github.workflow }}
+<% endif %>
   cancel-in-progress: true
 
 defaults:
@@ -226,7 +229,7 @@ jobs:
 
           if [[ -f ci/scripts/pre-test.sh ]]; then
             echo "Loading pre-test"
-            ./ci/scripts/pre-test.sh
+            ./ci/scripts/pre-test.sh ${{ matrix.platform }} ${{ matrix.arch }}
           fi
 
 <% if lang == "go" %>
@@ -393,7 +396,7 @@ jobs:
 
           if [[ -f ci/scripts/pre-test.sh ]]; then
             echo "Loading pre-test"
-            ./ci/scripts/pre-test.sh
+            ./ci/scripts/pre-test.sh ${{ matrix.platform }} ${{ matrix.arch }}
           fi
 
           docker ps
@@ -671,12 +674,16 @@ jobs:
           persist-credentials: false
 <% endif %>
       - name: load package
+<% if private or secrets["build:test"] %>
         env:
+<% if private %>
           COLUMNAR_CLOUD_API_TOKEN: ${{ secrets.COLUMNAR_CLOUD_API_TOKEN }}
+<% endif %>
 <% if secrets["build:test"] %>
 <% for name, val in secrets["build:test"].items() %>
           <{name}>: ${{ secrets.<{val}> }}
 <% endfor %>
+<% endif %>
 <% endif %>
         working-directory: <{ lang_subdir }>
         run: |
@@ -688,9 +695,9 @@ jobs:
           if [[ -f .env.ci ]]; then
             source .env.ci
           fi
-
+<% if private %>
           export COLUMNAR_DRIVER_LICENSE_KEY=$(curl --silent --fail -XGET -H 'Content-Type: application/json' -H "Authorization: Bearer $COLUMNAR_CLOUD_API_TOKEN" https://heimdall.columnar.tech/trial_license)
-
+<% endif %>
           pixi exec -s adbc-driver-manager -s pyarrow -s pytest python -m pytest -vs ci/test_package.py
 
   release:
diff --git a/adbc_drivers_dev/workflow.py b/adbc_drivers_dev/workflow.py
index 9ce9312..0ae0df8 100644
--- a/adbc_drivers_dev/workflow.py
+++ b/adbc_drivers_dev/workflow.py
@@ -134,7 +134,6 @@ def generate_workflows(args) -> int:
             f"{lang}_release.yaml",
             {
                 **params.to_dict(),
-                "pull_request_trigger_paths": [".github/workflows/go_release.yaml"],
                 "release": True,
                 "workflow_name": "Release",
                 "lang": lang,