Merge pull request #51 from localgovdrupal/4.x

4.0.0-beta1 release

.ddev/config.yaml

@@ -1,7 +1,7 @@
 name: localgov-microsites
 type: drupal10
 docroot: web
-php_version: "8.1"
+php_version: "8.2"
 webserver_type: nginx-fpm
 router_http_port: "80"
 router_https_port: "443"

.github/workflows/localgov_microsites.yml

@@ -6,10 +6,11 @@ name: Test LocalGov Microsites localgovdrupal/localgov_microsites_project projec
 on:
   push:
     branches:
-      - '3.x'
+      - '4.x'
   pull_request:
     branches:
-      - '3.x'
+      - '4.x'
+  workflow_dispatch:
 
 env:
   LOCALGOV_DRUPAL_PROJECT: localgovdrupal/localgov_microsites_project
@@ -25,7 +26,7 @@ jobs:
       fail-fast: false
       matrix:
         localgov-version:
-          - '3.x'
+          - '4.x'
         drupal-version:
           - '~10.0'
         php-version:
@@ -89,7 +90,7 @@ jobs:
       fail-fast: false
       matrix:
         localgov-version:
-          - '3.x'
+          - '4.x'
         drupal-version:
           - '~10.0'
         php-version:
@@ -125,7 +126,7 @@ jobs:
       fail-fast: false
       matrix:
         localgov-version:
-          - '3.x'
+          - '4.x'
         drupal-version:
           - '~10.0'
         php-version:
@@ -160,7 +161,7 @@ jobs:
       fail-fast: false
       matrix:
         localgov-version:
-          - '3.x'
+          - '4.x'
         drupal-version:
           - '~10.0'
         php-version:

.lando.dist.yml

@@ -2,7 +2,7 @@ name: localgov-micro
 recipe: drupal10
 config:
   webroot: web
-  php: '8.1'
+  php: '8.2'
 proxy:
   mailhog:
     - mail.localgov-micro.lndo.site
@@ -75,6 +75,9 @@ services:
         - '4444'
       command: ["/usr/local/bin/supervisord", "-c", "/etc/supervisord.conf"]
 tooling:
+  composer:
+    service: appserver
+    cmd: /usr/local/bin/composer
   deprecated:
     service: appserver
     cmd: 'bash -c "/app/bin/phpstan analyse -c /app/phpstan.neon /app/web/profiles/contrib/localgov_*/ /app/web/modules/contrib/localgov_*/  /app/web/themes/contrib/localgov_*"'

README.md

@@ -9,7 +9,7 @@ First you need to [install Composer](https://getcomposer.org/doc/00-intro.md#ins
 After that you can create the project:
 
 ```shell
-composer create-project --stability beta localgovdrupal/localgov_microsites_project MY_PROJECT  --no-install
+composer create-project --stability alpha localgovdrupal/localgov_microsites_project MY_PROJECT  --no-install
 ```
 
 ## Using Lando
@@ -33,7 +33,7 @@ composer install
 drush si localgov_microsites
 ```
 
-## Runnint PHPUnit tests in Lando
+## Running PHPUnit tests in Lando
 
 The included phpunit.xml.dist file contains configuration for automatically
 running the LocalGov Drupal test suite.

phpstan.neon

@@ -1,9 +1,13 @@
 parameters:
-	customRulesetUsed: true
-	reportUnmatchedIgnoredErrors: false
-	# Ignore phpstan-drupal extension's rules.
-	ignoreErrors:
-		- '#\Drupal calls should be avoided in classes, use dependency injection instead#'
-		- '#Plugin definitions cannot be altered.#'
-		- '#Missing cache backend declaration for performance.#'
-		- '#Plugin manager has cache backend specified but does not declare cache tags.#'
+  # we don't set a level so that we check deprecations only, and
+  # not to highlight unknown classes which are from composer suggestions.
+  customRulesetUsed: true
+  reportUnmatchedIgnoredErrors: false
+  # Ignore phpstan-drupal extension's rules.
+  ignoreErrors:
+    - '#\Drupal calls should be avoided in classes, use dependency injection instead#'
+    - '#Plugin definitions cannot be altered.#'
+    - '#Missing cache backend declaration for performance.#'
+    - '#Plugin manager has cache backend specified but does not declare cache tags.#'
+    # new static() is a best practice in Drupal, so we cannot fix that.
+    - '#^Unsafe usage of new static#'