Fix issues with new slevomat rules

Author: tvdijen

.github/workflows/php.yml

@@ -162,7 +162,7 @@ jobs:
         with:
           # Should be the higest supported version, so we can use the newest tools
           php-version: '8.4'
-          tools: composer, composer-require-checker, composer-unused, phpcs
+          tools: composer, composer-require-checker, composer-unused
           extensions: ctype, date, dom, filter, hash, intl, mbstring, opcache, openssl, pcre, spl, xml
 
       - name: Setup problem matchers for PHP
@@ -194,12 +194,13 @@ jobs:
           composer-unused \
           --excludePackage=slevomat/coding-standard \
           --excludePackage=symfony/phpunit-bridge \
+          --excludePackage=squizlabs/php_codesniffer \
           --excludePackage=phpstan/phpstan \
           --excludePackage=phpstan/extension-installer \
           --excludePackage=phpstan/phpstan-mockery
 
       - name: PHP Code Sniffer
-        run: phpcs
+        run: vendor/bin/phpcs
 
       - name: PHPStan
         run: |

composer.json

@@ -30,6 +30,7 @@
         "phpunit/phpunit": "^11.5 || ^12.3",
         "psr/log": "^3.0",
         "slevomat/coding-standard": "^8.20",
+        "squizlabs/php_codesniffer": "^3.13 || ^4.0",
         "symfony/phpunit-bridge": "^7.3 || ^8.0"
     },
     "require-dev": {

lib/ArrayLogger.php

@@ -17,16 +17,19 @@ class ArrayLogger implements LoggingHandlerInterface
      */
     public array $logs = [];
 
+
     public function __construct(Configuration $config)
     {
         // don't do anything with the configuration
     }
 
+
     public function log(int $level, string $string): void
     {
         $this->logs[$level][] = $string;
     }
 
+
     public function setLogFormat(string $format): void
     {
         // ignored