Merge pull request #19 from veewee/php84

Add PHP 8.4 support
veewee · Oct 25, 2024 · e5d015b · e5d015b
2 parents 33fff77 + 6191e43
commit e5d015b
Show file tree

Hide file tree

Showing 6 changed files with 12 additions and 6 deletions.
diff --git a/.github/workflows/analyzers.yaml b/.github/workflows/analyzers.yaml
@@ -12,7 +12,8 @@ jobs:
         strategy:
             matrix:
                 operating-system: [ubuntu-latest]
-                php-versions: ['8.1', '8.2', '8.3']
+                php-versions: ['8.1', '8.2', '8.3', '8.4']
+                composer-options: ['--ignore-platform-req=php+']
             fail-fast: false
         name: PHP ${{ matrix.php-versions }} @ ${{ matrix.operating-system }}
         steps:
@@ -28,3 +29,4 @@ jobs:
               run: composer update --ignore-platform-req=php+ --prefer-dist --no-progress --no-suggest ${{ matrix.composer-options }}
             - name: Run the tests
               run: composer run psalm
+              continue-on-error: ${{ matrix.php-versions == '8.4' }} # Infection dependency is causing issues.
diff --git a/.github/workflows/code-style.yaml b/.github/workflows/code-style.yaml
@@ -11,7 +11,8 @@ jobs:
         strategy:
             matrix:
                 operating-system: [ubuntu-latest]
-                php-versions: ['8.1', '8.2', '8.3']
+                php-versions: ['8.1', '8.2', '8.3', '8.4']
+                composer-options: ['--ignore-platform-req=php+']
             fail-fast: false
         name: PHP ${{ matrix.php-versions }} @ ${{ matrix.operating-system }}
         steps:

diff --git a/.github/workflows/tests.yaml b/.github/workflows/tests.yaml
@@ -12,7 +12,8 @@ jobs:
         strategy:
             matrix:
                 operating-system: [ubuntu-latest]
-                php-versions: ['8.1', '8.2', '8.3']
+                php-versions: ['8.1', '8.2', '8.3', '8.4']
+                composer-options: ['--ignore-platform-req=php+']
                 experimental: [false]
             fail-fast: false
         name: PHP ${{ matrix.php-versions }} @ ${{ matrix.operating-system }}
@@ -27,8 +28,9 @@ jobs:
                   ini-values: error_reporting=E_ALL
                   extensions: pcov, mbstring, posix
             - name: Install dependencies
-              run: composer update --prefer-dist --no-progress --no-suggest
+              run: composer update --prefer-dist --no-progress --no-suggest ${{ matrix.composer-options }}
             - name: Run the tests
               run: composer run tests
             - name: Check tests quality
               run: composer run testquality
+              continue-on-error: ${{ matrix.php-versions == '8.4' }} # Infection is not ready yet.
diff --git a/.php-cs-fixer.dist.php b/.php-cs-fixer.dist.php
@@ -54,5 +54,6 @@
         'strict_comparison' => true,
         'strict_param' => true,
         'native_function_invocation' => true,
+        'nullable_type_declaration_for_default_null_value' => true,
     ])
 ;
diff --git a/composer.json b/composer.json
@@ -46,7 +46,7 @@
         }
     ],
     "require": {
-        "php": "~8.1.0 || ~8.2.0 || ~8.3.0 ",
+        "php": "~8.1.0 || ~8.2.0 || ~8.3.0 || ~8.4.0",
         "azjezz/psl": "^2.7 || ^3.0"
     },
     "require-dev": {

diff --git a/src/Iso/object_data.php b/src/Iso/object_data.php
@@ -17,7 +17,7 @@
  *
  * @psalm-pure
  */
-function object_data(string $className, Lens $accessor = null): Iso
+function object_data(string $className, ?Lens $accessor = null): Iso
 {
     /** @var Lens<S, A> $typedAccessor */
     $typedAccessor = $accessor ?? properties();