New CI flow

Author: nanomad

…thub/workflows/build_and_test_python.yml .github/workflows/ci.yml.github/workflows/build_and_test_python.yml renamed to .github/workflows/ci.yml .github/workflows/build_and_test_python.yml renamed to .github/workflows/ci.yml

@@ -1,21 +1,31 @@
-name: Python tests
+name: CI
+run-name: "${{ github.event_name == 'workflow_dispatch' && format('CI: {0}', github.ref_name) || '' }}"
 
 on:
   push:
     branches:
       - '*'
   pull_request:
-    types: [ opened, synchronize, reopened ]
-    branches:
-      - '*'
+  release:
+    types: [ published ]
+  workflow_dispatch:
+
+concurrency:
+  group: ${{ github.workflow }}-${{ (github.event_name == 'release' && github.run_id) || github.event.pull_request.number || github.ref }}
+  cancel-in-progress: true
+
+permissions:
+  contents: read
 
 jobs:
   build:
 
     runs-on: ubuntu-latest
+
     strategy:
+      fail-fast: false
       matrix:
-        python-version: [ "3.11", "3.12" ]
+        python-version: [ "3.11", "3.12", "3.13" ]
 
     steps:
       - name: Checkout code
@@ -44,17 +54,22 @@ jobs:
         run: |
           poetry install --no-interaction --no-root
 
-      - name: Install library
+      - name: Install as a library
         run: poetry install --no-interaction
 
-      - name: Run tests with coverage
+      - name: Run mypy
         run: |
-          poetry run pytest tests --cov --junit-xml=junit/test-results-${{ matrix.python-version }}.xml
+          poetry run mypy src
+        # Temporary until we fix all typing issues
+        continue-on-error: true
 
       - name: Lint with Ruff
         run: |
           poetry run ruff check .
-        continue-on-error: true
+
+      - name: Run tests with coverage
+        run: |
+          poetry run pytest tests --cov --junit-xml=junit/test-results-${{ matrix.python-version }}.xml
 
       - name: Surface failing tests
         uses: pmeier/pytest-results-action@main
@@ -70,4 +85,32 @@ jobs:
 
           # (Optional) Fail the workflow if no JUnit XML was found.
           fail-on-empty: true
-        if: ${{ always() }}
+        if: ${{ always() }}
+
+  release:
+    name: Release to PyPi
+    if: ${{ github.event_name == 'release' }}
+    runs-on: ubuntu-latest
+    needs:
+      - build
+
+    environment:
+      name: release
+      url: https://pypi.org/p/saic_ismart_client_ng
+
+    permissions:
+      contents: read
+
+    steps:
+      - name: Check out code from GitHub
+        uses: actions/checkout@v4
+
+      - name: Set up Python
+        uses: actions/setup-python@v5
+        with:
+          python-version: '3.11'
+
+      - name: Build and publish to pypi
+        uses: JRubics/[email protected]
+        with:
+          pypi_token: ${{ secrets.PYPI_API_TOKEN }}

.github/workflows/python-publish.yml

@@ -80,3 +80,6 @@ ignore_errors = true
 
 [tool.ruff]
 output-format = "github"
+
+[tool.mypy]
+files = ["./src", "./tests"]

pyproject.toml

@@ -74,7 +74,7 @@ def encrypt_request(
     request_path = str(original_request_url).replace(base_uri, "/")
     request_body = original_request_content
     new_content = original_request_content
-    if request_body and "multipart" not in original_content_type:
+    if request_body and (not original_content_type or "multipart" not in original_content_type):
         modified_content_type = normalize_content_type(original_content_type)
         request_content = request_body.strip()
         if request_content: