chore: add phpstan strict type checking (#153)

* chore: add phpstan strict type checking

chore: update lando tooling to better reflect usage

chore: only run phpcs and phpstan on latest php version

chore: update step name

feat: update phpstan to mirro upgrade_status results

feat: update phpstan to ignore errors from phpunit dynamic call

fix: remove booleansInConditions check

* feat: added baseline

* fix: update baseline file

* feat: ignore Drupal 12 deprecations

* fix: update baseline

.github/workflows/test.yml

@@ -11,15 +11,14 @@ on:
     branches:
       - '3.x'
   schedule:
-    - cron: "0 7 * * *"
+    - cron: '0 7 * * *'
   workflow_dispatch:
 
 env:
   LOCALGOV_DRUPAL_PROJECT: localgovdrupal/localgov-project
   LOCALGOV_DRUPAL_PROJECT_PATH:
 
 jobs:
-
   build:
     name: Install LocalGov Drupal
     runs-on: ubuntu-latest
@@ -39,7 +38,6 @@ jobs:
           - '8.3'
 
     steps:
-
       - name: Save git branch and git repo names to env if this is not a pull request
         if: github.event_name != 'pull_request'
         run: |
@@ -98,12 +96,9 @@ jobs:
         drupal-version:
           - '~10.0'
         php-version:
-          - '8.1'
-          - '8.2'
           - '8.3'
 
     steps:
-
       - name: Cached workspace
         uses: actions/cache@v2
         with:
@@ -123,7 +118,7 @@ jobs:
           ./bin/phpcs -p ${LOCALGOV_DRUPAL_PROJECT_PATH}
 
   phpstan:
-    name: Deprecated code checks
+    name: Static analysis checks
     needs: build
     runs-on: ubuntu-latest
 
@@ -135,12 +130,9 @@ jobs:
         drupal-version:
           - '~10.0'
         php-version:
-          - '8.1'
-          - '8.2'
           - '8.3'
 
     steps:
-
       - name: Cached workspace
         uses: actions/cache@v2
         with:
@@ -154,7 +146,7 @@ jobs:
         with:
           php-version: ${{ matrix.php-version }}
 
-      - name: Run deprecated code checks
+      - name: Run PHPStan code checks
         run: |
           cd html
           ./bin/phpstan analyse -c ./phpstan.neon ./web/modules/contrib/localgov* ./web/profiles/contrib/localgov* ./web/themes/contrib/localgov*
@@ -176,7 +168,6 @@ jobs:
           - '8.3'
 
     steps:
-
       - name: Clone Drupal container
         uses: actions/checkout@v2
         with:

.lando.dist.yml

@@ -75,7 +75,7 @@ tooling:
   composer:
     service: appserver
     cmd: /usr/local/bin/composer
-  deprecated:
+  sca:
     service: appserver
     cmd: 'bash -c "/app/vendor/bin/phpstan analyse -c /app/phpstan.neon /app/web/profiles/contrib/localgov* /app/web/modules/contrib/localgov*  /app/web/themes/contrib/localgov*"'
   drush:

composer.json

@@ -26,6 +26,7 @@
         "phpspec/prophecy-phpunit": "^2",
         "phpstan/phpstan": "^1.0",
         "phpstan/phpstan-deprecation-rules": "^1.0",
+        "phpstan/phpstan-strict-rules": "^1.5",
         "squizlabs/php_codesniffer": "^3.6"
     },
     "conflict": {