Consolidate CI workflows; gate deploy on linter and tests

Merges php-linter.yml and php-tests.yml into a single ci.yml with
three jobs: linter, tests, and deploy. The deploy job uses
needs: [linter, tests] so it only runs on develop/phoenix pushes
after both checks pass.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: carlbennett

.github/workflows/deployment.yml .github/workflows/ci.yml.github/workflows/deployment.yml renamed to .github/workflows/ci.yml

@@ -3,19 +3,81 @@ name: deployment
 
 on:
   push:
-    branches:
-      - develop
-      - phoenix
+  pull_request:
   workflow_dispatch:
+    inputs:
+      php_version:
+        description: "Specify the PHP version to use (e.g., 8.3)"
+        required: false
+        default: '8.3'
 
 env:
-  PHP_VERSION: '8.3'
+  PHP_VERSION: ${{ inputs.php_version || '8.3' }}
 
 permissions:
   contents: read
 
 jobs:
+  linter:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+
+      - name: Update apt cache
+        run: sudo apt-get update
+
+      - name: Install PHP ${{ env.PHP_VERSION }}
+        run: sudo apt-get install php${{ env.PHP_VERSION }}-cli php${{ env.PHP_VERSION }}-mbstring php${{ env.PHP_VERSION }}-xml
+
+      - name: Validate composer.json and composer.lock
+        run: composer validate --strict
+
+      - name: Cache Composer packages
+        id: composer-cache
+        uses: actions/cache@v4
+        with:
+          path: lib
+          key: ${{ runner.os }}-php-${{ hashFiles('**/composer.lock') }}
+          restore-keys: |
+            ${{ runner.os }}-php-
+
+      - name: Install composer/vendor dependencies
+        run: composer install --prefer-dist --no-progress --ignore-platform-reqs
+
+      - name: Validate PHP syntax
+        run: bash -c 'set -e;for file in $(find ./src ./tests -type f -regex ".*\.\(php\|phtml\)" -print); do php -e -l -f "$file"; done'
+
+  tests:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+
+      - name: Update apt cache
+        run: sudo apt-get update
+
+      - name: Install PHP ${{ env.PHP_VERSION }}
+        run: sudo apt-get install php${{ env.PHP_VERSION }}-cli php${{ env.PHP_VERSION }}-mbstring php${{ env.PHP_VERSION }}-xml
+
+      - name: Cache Composer packages
+        id: composer-cache
+        uses: actions/cache@v4
+        with:
+          path: lib
+          key: ${{ runner.os }}-php-${{ hashFiles('**/composer.lock') }}
+          restore-keys: |
+            ${{ runner.os }}-php-
+
+      - name: Install composer/vendor dependencies
+        run: composer install --prefer-dist --no-progress --ignore-platform-reqs
+
+      - name: Run PHPUnit tests
+        run: lib/bin/phpunit --testdox
+
   deploy:
+    needs: [linter, tests]
+    if: github.event_name == 'push' && (github.ref == 'refs/heads/develop' || github.ref == 'refs/heads/phoenix')
     runs-on: ubuntu-latest
     environment: production
     steps:
@@ -31,9 +93,6 @@ jobs:
       - name: Install PHP ${{ env.PHP_VERSION }}
         run: sudo apt-get install php${{ env.PHP_VERSION }}-cli php${{ env.PHP_VERSION }}-mbstring php${{ env.PHP_VERSION }}-xml
 
-      - name: Validate composer.json and composer.lock
-        run: composer validate --strict
-
       - name: Cache Composer packages
         id: composer-cache
         uses: actions/cache@v4
@@ -46,9 +105,6 @@ jobs:
       - name: Install composer/vendor dependencies
         run: composer install --prefer-dist --no-progress --ignore-platform-reqs
 
-      - name: Validate PHP syntax
-        run: bash -c 'set -e;for file in $(find ./src -type f -regex ".*\.\(php\|phtml\)" -print); do php -e -l -f "$file"; done'
-
       - name: Sanitize Branch Name
         run: |
           CLEAN_BRANCH_NAME="${GITHUB_REF_NAME//\//_}"