Build and Test Package Before Creating Release (#1645)

* update build+test workflow to run on push and pr

* test on all supported python versions

* remove pip-install job from python-checks

* use package requirements file to enable pip cache

* avoid platform-specific tar flags

Author: bmos

.github/workflows/python-checks.yml

@@ -1,14 +1,14 @@
 name: Python checks
 
 on:
+  workflow_dispatch:
+  pull_request:
   push:
     branches:
       - main
       - major-release
-  pull_request:
   schedule:
     - cron: 0 0 * * 1
-  workflow_dispatch:
 
 permissions: read-all
 
@@ -286,45 +286,3 @@ jobs:
           pre-commit run --all-files
         env:
           PRE_COMMIT_HOME: .pre-commit-cache
-
-  pip-install:
-    strategy:
-      fail-fast: false
-      matrix:
-        python-version:
-          - '3.9'
-          - '3.10'
-          - '3.11'
-          - '3.12'
-          - '3.13'
-          - '3.14'
-        os:
-          - ubuntu-latest
-          - windows-latest
-          - macos-latest
-        limited-dependencies:
-          - 'TRUE'
-          - ''
-
-    runs-on: ${{ matrix.os }}
-
-    steps:
-      - uses: step-security/harden-runner@95d9a5deda9de15063e7595e9719c11c38c90ae2
-        with:
-          disable-sudo: true
-          egress-policy: audit
-
-      - uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8
-
-      - uses: actions/setup-python@83679a892e2d95755f2dac6acb0bfd1e9ac5d548
-        with:
-          python-version: ${{ matrix.python-version }}
-          cache: pip
-
-      - continue-on-error: ${{ contains(matrix.python-version, '3.14') }}
-        env:
-          PARSONS_LIMITED_DEPENDENCIES: ${{ matrix.limited-dependencies }}
-        run: |
-          pip install -r requirements-dev.txt
-          pip install -e .[all]
-

.github/workflows/release.yml

@@ -1,7 +1,12 @@
-name: Build & Publish Wheels + Source
+name: Build, test, & publish
 
 on:
   workflow_dispatch:
+  pull_request:
+  push:
+    branches:
+      - main
+      - major-release
   release:
     types:
       - published
@@ -45,10 +50,20 @@ jobs:
 
   test:
     needs: build
+
     strategy:
       fail-fast: false
       matrix:
-        os: [ubuntu-latest, windows-latest, macos-latest]
+        python-version:
+          - '3.9'
+          - '3.10'
+          - '3.11'
+          - '3.12'
+          - '3.13'
+        os:
+          - ubuntu-latest
+          - windows-latest
+          - macos-latest
 
     runs-on: ${{ matrix.os }}
 
@@ -59,16 +74,33 @@ jobs:
           name: parsons-dist
           path: dist
 
-      - name: Install package using pip
+      - name: Extract requirements for cache
+        shell: bash
         run: |
           file=$(find ./dist -name 'parsons-*.tar.gz' | head -1)
-          pip install "$file"
+          # Extract to a temp directory first
+          mkdir -p temp_extract
+          tar -xzf "$file" -C temp_extract
+          # Find and move requirements files to root
+          find temp_extract -name 'requirements.txt' -exec cp {} . \; 2>/dev/null || true
+          # Clean up
+          rm -rf temp_extract
+
+      - name: Set up Python ${{ matrix.python-version }}
+        uses: actions/setup-python@e797f83bcb11b83ae66e0230d6156d7c80228e7c
+        with:
+            python-version: ${{ matrix.python-version }}
+            cache: pip
+
+      - name: Install package
         shell: bash
+        run: |
+          file=$(find ./dist -name 'parsons-*.tar.gz' | head -1)
+          pip install "$file"
 
   publish:
-    if: github.repository_owner == 'move-coop'
-    needs:
-      - test
+    if: github.repository_owner == 'move-coop' && github.event_name == 'release' && github.event.action == 'published'
+    needs: test
 
     runs-on: ubuntu-latest
     environment: release
@@ -85,7 +117,6 @@ jobs:
           name: parsons-dist
           path: dist
 
-      # Publish
       - name: Publish
         uses: pypa/gh-action-pypi-publish@ed0c53931b1dc9bd32cbe73a98c7f6766f8a527e
         with: