PHP 8.4 support (#390)

Author: vjik

.github/workflows/bc.yml

@@ -30,4 +30,4 @@ jobs:
       os: >-
         ['ubuntu-latest']
       php: >-
-        ['8.1']
+        ['8.4']

.github/workflows/build.yml

@@ -31,4 +31,4 @@ jobs:
       os: >-
         ['ubuntu-latest', 'windows-latest']
       php: >-
-        ['8.1', '8.2', '8.3']
+        ['8.1', '8.2', '8.3', '8.4']

.github/workflows/composer-require-checker.yml

@@ -31,4 +31,4 @@ jobs:
       os: >-
         ['ubuntu-latest']
       php: >-
-        ['8.1', '8.2', '8.3']
+        ['8.1', '8.2', '8.3', '8.4']

.github/workflows/rector.yml

@@ -21,4 +21,4 @@ jobs:
       os: >-
         ['ubuntu-latest']
       php: >-
-        ['8.3']
+        ['8.4']

.github/workflows/static.yml

@@ -29,4 +29,4 @@ jobs:
       os: >-
         ['ubuntu-latest']
       php: >-
-        ['8.1', '8.2', '8.3']
+        ['8.1', '8.2', '8.3', '8.4']

CHANGELOG.md

@@ -6,6 +6,8 @@
 - Enh #384: Make `$config` parameter in `Container` constructor optional (@np25071984)
 - Enh #324: Make `BuildingException` and `NotFoundException` friendly (@np25071984)
 - Enh #387: Improve container performance (@samdark)
+- Chg #390: Change PHP constraint in `composer.json` to `8.1 - 8.4` (@vjik)
+- Bug #390: Explicitly mark nullable parameters (@vjik)
 
 ## 1.3.0 October 14, 2024
 

composer.json

@@ -32,23 +32,23 @@
         }
     ],
     "require": {
-        "php": "^8.1",
+        "php": "8.1 - 8.4",
         "ext-mbstring": "*",
-        "psr/container": "^1.1|^2.0",
+        "psr/container": "^1.1 || ^2.0",
         "yiisoft/definitions": "^3.0",
         "yiisoft/friendly-exception": "^1.1.0"
     },
     "require-dev": {
-        "league/container": "^5.0",
-        "maglnet/composer-require-checker": "^4.2",
-        "phpbench/phpbench": "^1.2.0",
-        "phpunit/phpunit": "^10.5",
-        "rector/rector": "^2.0",
-        "roave/infection-static-analysis-plugin": "^1.25",
-        "spatie/phpunit-watcher": "^1.23",
-        "vimeo/psalm": "^5.26.1",
-        "yiisoft/injector": "^1.0",
-        "yiisoft/test-support": "^3.0"
+        "league/container": "^5.0.1",
+        "maglnet/composer-require-checker": "^4.7.1",
+        "phpbench/phpbench": "^1.4.1",
+        "phpunit/phpunit": "^10.5.45",
+        "rector/rector": "^2.0.11",
+        "roave/infection-static-analysis-plugin": "^1.35",
+        "spatie/phpunit-watcher": "^1.24",
+        "vimeo/psalm": "^5.26.1 || ^6.10",
+        "yiisoft/injector": "^1.2",
+        "yiisoft/test-support": "^3.0.2"
     },
     "suggest": {
         "yiisoft/injector": "^1.0",

psalm.xml

@@ -3,6 +3,7 @@
     errorLevel="1"
     findUnusedBaselineEntry="true"
     findUnusedCode="false"
+    ensureOverrideAttribute="false"
     xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
     xmlns="https://getpsalm.org/schema/config"
     xsi:schemaLocation="https://getpsalm.org/schema/config vendor/vimeo/psalm/config.xsd"

src/BuildingException.php

@@ -23,7 +23,7 @@ public function __construct(
         private readonly string $id,
         Throwable $error,
         array $buildStack = [],
-        Throwable $previous = null,
+        ?Throwable $previous = null,
     ) {
         $message = sprintf(
             'Caught unhandled error "%s" while building "%s".',

tests/Support/EngineFactory.php

@@ -16,7 +16,7 @@ public function __construct(private readonly ContainerInterface $container)
     {
     }
 
-    public function createByName(string $name = null): EngineInterface
+    public function createByName(?string $name = null): EngineInterface
     {
         if ($name === EngineMarkOne::NAME) {
             return $this->container->get(EngineMarkOne::class);