#459 Migrate workflows to centralized SDK version matrix and fix job outputs (#460)

* #459 Migrate workflows to centralized SDK version matrix and fix job outputs

* fix: formatting and zizmor template-injection warnings

Author: kernelsam

.github/workflows/build-distribution.yaml

@@ -8,8 +8,6 @@ permissions:
 jobs:
   build-distribution:
     name: Build distribution
-    outputs:
-      status: ${{ job.status }}
     permissions:
       contents: read
     runs-on: ubuntu-latest
@@ -51,10 +49,10 @@ jobs:
 
   slack-notification:
     needs: [build-distribution]
-    if: ${{ always() && contains(fromJSON('["failure", "cancelled"]'), needs.build-distribution.outputs.status ) && github.ref_name == github.event.repository.default_branch }}
+    if: ${{ always() && contains(fromJSON('["failure", "cancelled"]'), needs.build-distribution.result ) && github.ref_name == github.event.repository.default_branch }}
     secrets:
       SLACK_BOT_TOKEN: ${{ secrets.SLACK_BOT_TOKEN }}
       SLACK_CHANNEL: ${{ secrets.SLACK_CHANNEL }}
     uses: senzing-factory/build-resources/.github/workflows/build-failure-slack-notification.yaml@v4
     with:
-      job-status: ${{ needs.build-distribution.outputs.status }}
+      job-status: ${{ needs.build-distribution.result }}

.github/workflows/create-sphinx-documentation.yaml

@@ -10,8 +10,6 @@ permissions: {}
 
 jobs:
   docs:
-    outputs:
-      status: ${{ job.status }}
     permissions:
       contents: write
     runs-on: ubuntu-latest
@@ -52,10 +50,10 @@ jobs:
 
   slack-notification:
     needs: [docs]
-    if: ${{ always() && contains(fromJSON('["failure", "cancelled"]'), needs.docs.outputs.status ) }}
+    if: ${{ always() && contains(fromJSON('["failure", "cancelled"]'), needs.docs.result ) }}
     secrets:
       SLACK_BOT_TOKEN: ${{ secrets.SLACK_BOT_TOKEN }}
       SLACK_CHANNEL: ${{ secrets.SLACK_CHANNEL }}
     uses: senzing-factory/build-resources/.github/workflows/build-failure-slack-notification.yaml@v4
     with:
-      job-status: ${{ needs.docs.outputs.status }}
+      job-status: ${{ needs.docs.result }}

.github/workflows/publish-to-pypi.yaml

@@ -10,8 +10,6 @@ permissions: {}
 jobs:
   build-distribution:
     name: Build distribution
-    outputs:
-      status: ${{ job.status }}
     permissions:
       contents: read
     runs-on: ubuntu-latest
@@ -54,8 +52,6 @@ jobs:
     name: Publish Python distribution to PyPI
     needs:
       - build-distribution
-    outputs:
-      status: ${{ job.status }}
     permissions:
       id-token: write
     runs-on: ubuntu-latest
@@ -74,8 +70,6 @@ jobs:
     name: Sign the Python distribution with Sigstore and upload them to GitHub Release
     needs:
       - publish-to-pypi
-    outputs:
-      status: ${{ job.status }}
     permissions:
       contents: write # IMPORTANT: mandatory for making GitHub Releases
       id-token: write # IMPORTANT: mandatory for sigstore
@@ -105,10 +99,10 @@ jobs:
 
   slack-notification:
     needs: [build-distribution, publish-to-pypi, github-release]
-    if: ${{ always() && contains(fromJSON('["failure", "cancelled"]'), needs.build-distribution.outputs.status ) && contains(fromJSON('["failure", "cancelled"]'), needs.publish-to-pypi.outputs.status ) && contains(fromJSON('["failure", "cancelled"]'), needs.github-release.outputs.status ) && github.ref_name == github.event.repository.default_branch }}
+    if: ${{ always() && contains(fromJSON('["failure", "cancelled"]'), needs.build-distribution.result ) && contains(fromJSON('["failure", "cancelled"]'), needs.publish-to-pypi.result ) && contains(fromJSON('["failure", "cancelled"]'), needs.github-release.result ) && github.ref_name == github.event.repository.default_branch }}
     secrets:
       SLACK_BOT_TOKEN: ${{ secrets.SLACK_BOT_TOKEN }}
       SLACK_CHANNEL: ${{ secrets.SLACK_CHANNEL }}
     uses: senzing-factory/build-resources/.github/workflows/build-failure-slack-notification.yaml@v4
     with:
-      job-status: ${{ needs.build-distribution.outputs.status && needs.publish-to-pypi.outputs.status && needs.github-release.outputs.status }}
+      job-status: ${{ needs.build-distribution.result && needs.publish-to-pypi.result && needs.github-release.result }}

.github/workflows/pylint.yaml

@@ -12,8 +12,6 @@ permissions: {}
 
 jobs:
   pylint:
-    outputs:
-      status: ${{ job.status }}
     permissions:
       contents: read
     runs-on: ubuntu-latest
@@ -49,10 +47,10 @@ jobs:
 
   slack-notification:
     needs: [pylint]
-    if: ${{ always() && contains(fromJSON('["failure", "cancelled"]'), needs.pylint.outputs.status ) && github.ref_name == github.event.repository.default_branch }}
+    if: ${{ always() && contains(fromJSON('["failure", "cancelled"]'), needs.pylint.result ) && github.ref_name == github.event.repository.default_branch }}
     secrets:
       SLACK_BOT_TOKEN: ${{ secrets.SLACK_BOT_TOKEN }}
       SLACK_CHANNEL: ${{ secrets.SLACK_CHANNEL }}
     uses: senzing-factory/build-resources/.github/workflows/build-failure-slack-notification.yaml@v4
     with:
-      job-status: ${{ needs.pylint.outputs.status }}
+      job-status: ${{ needs.pylint.result }}

.github/workflows/pytest-darwin.yaml

@@ -18,8 +18,17 @@ env:
 permissions: {}
 
 jobs:
+  sdk-versions:
+    runs-on: ubuntu-latest
+    outputs:
+      senzingsdk-versions: ${{ steps.cfg.outputs.senzingsdk-versions }}
+    steps:
+      - id: cfg
+        uses: senzing-factory/build-resources/sdk-versions@v4
+
   pytest-darwin:
     name: "Pytest with Senzing: ${{ matrix.senzingsdk-version }}; OS: ${{ matrix.os }}; Python ${{ matrix.python-version }}"
+    needs: sdk-versions
     permissions:
       contents: read
     runs-on: ${{ matrix.os }}
@@ -28,7 +37,7 @@ jobs:
       matrix:
         os: [macos-latest]
         python-version: ["3.10", "3.11", "3.12", "3.13"]
-        senzingsdk-version: [production-v4, staging-v4]
+        senzingsdk-version: ${{ fromJSON(needs.sdk-versions.outputs.senzingsdk-versions) }}
     timeout-minutes: 30
 
     steps:

.github/workflows/pytest-linux.yaml

@@ -20,8 +20,17 @@ env:
 permissions: {}
 
 jobs:
+  sdk-versions:
+    runs-on: ubuntu-latest
+    outputs:
+      senzingsdk-versions: ${{ steps.cfg.outputs.senzingsdk-versions }}
+    steps:
+      - id: cfg
+        uses: senzing-factory/build-resources/sdk-versions@v4
+
   pytest-linux:
     name: "Pytest with Senzing: ${{ matrix.senzingsdk-version }}; OS: ${{ matrix.os }}; Python ${{ matrix.python-version }}"
+    needs: sdk-versions
     permissions:
       contents: read
     runs-on: ${{ matrix.os }}
@@ -30,7 +39,7 @@ jobs:
       matrix:
         os: [ubuntu-latest]
         python-version: ["3.10", "3.11", "3.12", "3.13"]
-        senzingsdk-version: [production-v4, staging-v4]
+        senzingsdk-version: ${{ fromJSON(needs.sdk-versions.outputs.senzingsdk-versions) }}
     timeout-minutes: 30
 
     steps:

.github/workflows/pytest-windows.yaml

@@ -18,16 +18,25 @@ env:
 permissions: {}
 
 jobs:
+  sdk-versions:
+    runs-on: ubuntu-latest
+    outputs:
+      senzingsdk-versions: ${{ steps.cfg.outputs.senzingsdk-versions }}
+    steps:
+      - id: cfg
+        uses: senzing-factory/build-resources/sdk-versions@v4
+
   pytest-windows:
     name: "Pytest with Senzing: ${{ matrix.senzingsdk-version }}; OS: windows-latest; Python ${{ matrix.python-version }}"
+    needs: sdk-versions
     permissions:
       contents: read
     runs-on: windows-latest
     strategy:
       fail-fast: false
       matrix:
         python-version: ["3.10", "3.11", "3.12", "3.13"]
-        senzingsdk-version: [production-v4, staging-v4]
+        senzingsdk-version: ${{ fromJSON(needs.sdk-versions.outputs.senzingsdk-versions) }}
     timeout-minutes: 30
 
     steps: