From 9ad3976e079a71dfc86d19a329cb05deccea8f09 Mon Sep 17 00:00:00 2001 From: Robert Gruen Date: Thu, 1 Jan 2026 17:41:39 -0800 Subject: [PATCH 1/6] moved check and format to their own jobs --- .github/workflows/ci.yml | 80 ++++++++++++++++++++++++++++++++++++++-- 1 file changed, 77 insertions(+), 3 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 3f89c78..a64fac4 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -3,7 +3,7 @@ name: CI on: push: - branches: [ "main" ] + branches: [ "main", "dev/robgruen/workflow_update3" ] pull_request_target: branches: [ "main" ] workflow_dispatch: # manual run @@ -44,9 +44,83 @@ jobs: echo "${{ github.triggering_actor }} does not have permissions on this repo." echo "Current permission level is ${{ steps.checkAccess.outputs.user-permission }}" echo "Job originally triggered by ${{ github.actor }}" - exit 1 + exit 1 - ci: + check: + strategy: + fail-fast: false + matrix: + os: [ubuntu-latest, windows-latest] + python-version: ['3.12', '3.13', '3.14'] + exclude: + - os: windows-latest + python-version: '3.13' + - os: windows-latest + python-version: '3.14' + runs-on: ${{ matrix.os }} + name: ${{ matrix.os }} check (py ${{ matrix.python-version }}) + + steps: + - uses: actions/checkout@v4 + with: + ref: ${{ github.event.pull_request.head.sha || github.ref }} + + - name: Set up Python + uses: actions/setup-python@v5 + with: + python-version: ${{ matrix.python-version }} + + - name: Install uv + uses: astral-sh/setup-uv@v7 + + - name: Install Dependencies + shell: bash + run: | + uv sync --extra dev + + - name: Run Check + shell: bash + run: | + uv run pyright src tests tools examples + + format: + strategy: + fail-fast: false + matrix: + os: [ubuntu-latest, windows-latest] + python-version: ['3.12', '3.13', '3.14'] + exclude: + - os: windows-latest + python-version: '3.13' + - os: windows-latest + python-version: '3.14' + runs-on: ${{ matrix.os }} + name: ${{ matrix.os }} format (py ${{ matrix.python-version }}) + + steps: + - uses: actions/checkout@v4 + with: + ref: ${{ github.event.pull_request.head.sha || github.ref }} + + - name: Set up Python + uses: actions/setup-python@v5 + with: + python-version: ${{ matrix.python-version }} + + - name: Install uv + uses: astral-sh/setup-uv@v7 + + - name: Install Dependencies + shell: bash + run: | + uv sync --extra dev + + - name: Run Format + shell: bash + run: | + uv run black -tpy312 -tpy313 -tpy314 src tests tools examples --check + + test: needs: permissions-check environment: name: build-pipeline From 89bb9ab8436b28d8dfe23c04bba342b33f442050 Mon Sep 17 00:00:00 2001 From: Robert Gruen Date: Thu, 1 Jan 2026 17:42:56 -0800 Subject: [PATCH 2/6] removed check and format --- .github/workflows/ci.yml | 16 ---------------- 1 file changed, 16 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index a64fac4..4a08f4f 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -129,7 +129,6 @@ jobs: matrix: os: [ubuntu-latest, windows-latest] python-version: ['3.12', '3.13', '3.14'] - task: [check, test, format] exclude: - os: windows-latest python-version: '3.13' @@ -156,20 +155,7 @@ jobs: run: | uv sync --extra dev - - name: Run Check - shell: bash - if: matrix.task == 'check' - run: | - uv run pyright src tests tools examples - - - name: Run Format - shell: bash - if: matrix.task == 'format' - run: | - uv run black -tpy312 -tpy313 -tpy314 src tests tools examples --check - - name: Login to Azure - if: matrix.task == 'test' uses: azure/login@v2.2.0 with: client-id: ${{ secrets.AZURE_CLIENTID }} @@ -177,13 +163,11 @@ jobs: subscription-id: ${{ secrets.AZURE_SUBSCRIPTIONID }} - name: Get Keys - if: matrix.task == 'test' run: | uv run python tools/get_keys.py --vault build-pipeline-kv - name: Run Test shell: bash - if: matrix.task == 'test' run: | uv run pytest From 5cc0f36bac56cfbb7be4d8e3e87445cfb3cbdb58 Mon Sep 17 00:00:00 2001 From: Robert Gruen Date: Sat, 3 Jan 2026 16:54:20 -0800 Subject: [PATCH 3/6] updated to use make. --- .github/workflows/ci.yml | 20 ++++++++++++++++---- 1 file changed, 16 insertions(+), 4 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 4a08f4f..fde2b99 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -78,10 +78,16 @@ jobs: run: | uv sync --extra dev - - name: Run Check + - name: Run Check (Linux) + if: runner.os == 'Linux' shell: bash run: | - uv run pyright src tests tools examples + make check + + - name: Run Check (Windows) + if: runner.os == 'Windows' + run: | + ./make.bat check format: strategy: @@ -115,10 +121,16 @@ jobs: run: | uv sync --extra dev - - name: Run Format + - name: Run Format (Linux) + if: runner.os == 'Linux' shell: bash run: | - uv run black -tpy312 -tpy313 -tpy314 src tests tools examples --check + make format + + - name: Run Format (Windows) + if: runner.os == 'Windows' + run: | + ./make.bat format test: needs: permissions-check From fab30b35635a25c8d3aaf6f505c672b3b714c565 Mon Sep 17 00:00:00 2001 From: Robert Gruen Date: Sat, 3 Jan 2026 16:54:39 -0800 Subject: [PATCH 4/6] update branch name for testing --- .github/workflows/ci.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index fde2b99..bbe05de 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -3,7 +3,7 @@ name: CI on: push: - branches: [ "main", "dev/robgruen/workflow_update3" ] + branches: [ "main", "dev/robgruen/workflow_update5" ] pull_request_target: branches: [ "main" ] workflow_dispatch: # manual run From d399c7431e8c30c802a6173dcae01719c9465218 Mon Sep 17 00:00:00 2001 From: Robert Gruen Date: Sat, 3 Jan 2026 16:59:16 -0800 Subject: [PATCH 5/6] made online/offline tests --- .github/workflows/ci.yml | 46 +++++++++++++++++++++++++++++++++------- 1 file changed, 38 insertions(+), 8 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index bbe05de..9f066a4 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -132,22 +132,52 @@ jobs: run: | ./make.bat format - test: - needs: permissions-check - environment: - name: build-pipeline + offline-test: strategy: fail-fast: false matrix: os: [ubuntu-latest, windows-latest] - python-version: ['3.12', '3.13', '3.14'] + python-version: ['3.12', '3.13'] exclude: - os: windows-latest python-version: '3.13' - - os: windows-latest - python-version: '3.14' runs-on: ${{ matrix.os }} - name: ${{ matrix.os }} ${{ matrix.task }} (py ${{ matrix.python-version }}) + name: ${{ matrix.os }} (py ${{ matrix.python-version }}) + + steps: + - uses: actions/checkout@v4 + with: + ref: ${{ github.event.pull_request.head.sha || github.ref }} + + - name: Set up Python + uses: actions/setup-python@v5 + with: + python-version: ${{ matrix.python-version }} + + - name: Install uv + uses: astral-sh/setup-uv@v7 + + - name: Install Dependencies + shell: bash + run: | + uv sync --extra dev + + - name: Run Test + shell: bash + run: | + uv run pytest + + online-test: + needs: permissions-check + environment: + name: build-pipeline + strategy: + fail-fast: false + matrix: + os: [ubuntu-latest] + python-version: ['3.14'] + runs-on: ${{ matrix.os }} + name: ${{ matrix.os }} (py ${{ matrix.python-version }}) steps: - uses: actions/checkout@v4 From 3f48898de413051d181fd39bd9a70065a706006e Mon Sep 17 00:00:00 2001 From: Robert Gruen Date: Sat, 3 Jan 2026 17:03:25 -0800 Subject: [PATCH 6/6] removed testing branch --- .github/workflows/ci.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 9f066a4..fc71d95 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -3,7 +3,7 @@ name: CI on: push: - branches: [ "main", "dev/robgruen/workflow_update5" ] + branches: [ "main" ] pull_request_target: branches: [ "main" ] workflow_dispatch: # manual run