Add PHP 8.4 to CI (#167)

Author: vjik

.github/workflows/build.yml

@@ -29,4 +29,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/mutation.yml

@@ -27,6 +27,6 @@ jobs:
       os: >-
         ['ubuntu-latest']
       php: >-
-        ['8.2']
+        ['8.4']
     secrets:
       STRYKER_DASHBOARD_API_KEY: ${{ secrets.STRYKER_DASHBOARD_API_KEY }}

.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

@@ -8,6 +8,7 @@
 - Enh #165: Raise the minimum PHP version to 8.1 (@vjik)
 - Bug #166: `ArrayHelper::getValue()` returns default value on empty array key (@vjik) 
 - Bug #166: `ArrayHelper::keyExists()` returns false on empty array key (@vjik) 
+- Chg #167: Change PHP constraint in `composer.json` to `~8.1.0 || ~8.2.0 || ~8.3.0 || ~8.4.0` (@vjik)
 
 ## 3.1.0 April 04, 2024
 

composer.json

@@ -28,16 +28,16 @@
         }
     ],
     "require": {
-        "php": "^8.1",
+        "php": "~8.1.0 || ~8.2.0 || ~8.3.0 || ~8.4.0",
         "yiisoft/strings": "^2.1"
     },
     "require-dev": {
-        "maglnet/composer-require-checker": "^4.7.1",
+        "bamarni/composer-bin-plugin": "^1.8.2",
         "phpunit/phpunit": "^10.5.43",
         "rector/rector": "^2.0.7",
         "roave/infection-static-analysis-plugin": "^1.35",
         "spatie/phpunit-watcher": "^1.24",
-        "vimeo/psalm": "^5.26.1"
+        "vimeo/psalm": "^5.26.1|^6.1"
     },
     "autoload": {
         "psr-4": {
@@ -53,10 +53,18 @@
         "sort-packages": true,
         "bump-after-update": "dev",
         "allow-plugins": {
+            "bamarni/composer-bin-plugin": true,
             "infection/extension-installer": true,
             "composer/package-versions-deprecated": true
         }
     },
+    "extra": {
+        "bamarni-bin": {
+            "bin-links": true,
+            "target-directory": "tools",
+            "forward-command": true
+        }
+    },
     "scripts": {
         "test": "phpunit --testdox --no-interaction",
         "test-watch": "phpunit-watcher watch"

src/ArrayHelper.php

@@ -217,7 +217,6 @@ public static function getValue(
             if (empty($key)) {
                 return $default;
             }
-            /** @psalm-var array<mixed,string|int> $key */
             $lastKey = array_pop($key);
             foreach ($key as $keyPart) {
                 $array = self::getRootValue($array, $keyPart, null);
@@ -951,6 +950,8 @@ public static function keyExists(array $array, array|float|int|string $key, bool
                 return self::rootKeyExists($array, end($key), $caseSensitive);
             }
 
+            /** @psalm-var non-empty-array<array-key,float|int|string> $key */
+
             foreach (self::getExistsKeys($array, array_shift($key), $caseSensitive) as $existKey) {
                 $array = self::getRootValue($array, $existKey, null);
                 if (is_array($array) && self::keyExists($array, $key, $caseSensitive)) {

tests/ArrayHelper/GetValueTest.php

@@ -5,7 +5,6 @@
 namespace Yiisoft\Arrays\Tests\ArrayHelper;
 
 use ArrayObject;
-use Error;
 use InvalidArgumentException;
 use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;

tools/.gitignore

@@ -0,0 +1,2 @@
+/*/vendor
+/*/composer.lock