fix: remove deprecation notices in php 8.4

Author: lindyhopchris

.github/workflows/tests.yml

@@ -14,7 +14,7 @@ jobs:
     strategy:
       fail-fast: true
       matrix:
-        php: [8.2, 8.3]
+        php: [8.2, 8.3, 8.4]
         laravel: [11]
 
     steps:

CHANGELOG.md

@@ -5,6 +5,10 @@ All notable changes to this project will be documented in this file. This projec
 
 ## Unreleased
 
+### Fixed
+
+- Remove deprecation notices for PHP 8.4.
+
 ## [3.0.0] - 2024-03-12
 
 ### Changed

composer.json

@@ -27,8 +27,8 @@
         "ext-json": "*",
         "illuminate/contracts": "^11.0",
         "illuminate/pipeline": "^11.0",
-        "laravel-json-api/core": "^4.0",
-        "laravel-json-api/validation": "^4.0"
+        "laravel-json-api/core": "^4.3.2",
+        "laravel-json-api/validation": "^4.2"
     },
     "require-dev": {
         "laravel-json-api/testing": "^3.0",

phpunit.xml

@@ -13,6 +13,7 @@
          failOnWarning="true"
          failOnDeprecation="true"
          failOnNotice="true"
+         displayDetailsOnTestsThatTriggerDeprecations="true"
 >
     <coverage/>
     <testsuites>

src/Pipes/ValidationExceptionHandler.php

@@ -36,7 +36,7 @@ class ValidationExceptionHandler
      */
     public function __construct(
         private readonly Factory $factory,
-        Translator $translator = null,
+        ?Translator $translator = null,
     ) {
         $this->translator = $translator;
     }