Merge pull request #18 from doyouhaobaby/master

update for php8.4

Author: oscarotero

.github/workflows/test.yml

@@ -19,6 +19,7 @@ jobs:
           - 8.1
           - 8.2
           - 8.3
+          - 8.4
       fail-fast: false
 
     steps:

src/PhpFunctionsScanner.php

@@ -12,7 +12,7 @@ class PhpFunctionsScanner implements FunctionsScannerInterface
     protected Parser $parser;
     protected ?array $validFunctions;
 
-    public function __construct(array $validFunctions = null, Parser $parser = null)
+    public function __construct(?array $validFunctions = null, ?Parser $parser = null)
     {
         $this->validFunctions = $validFunctions;
         $this->parser = $parser ?: (new ParserFactory())->createForNewestSupportedVersion();

src/PhpNodeVisitor.php

@@ -21,7 +21,7 @@ class PhpNodeVisitor implements NodeVisitor
     /** @var Comment[] */
     protected array $bufferComments = [];
 
-    public function __construct(string $filename, array $validFunctions = null)
+    public function __construct(string $filename, ?array $validFunctions = null)
     {
         $this->filename = $filename;
         $this->validFunctions = $validFunctions;

src/PhpScanner.php

@@ -43,7 +43,7 @@ protected function saveTranslation(
         ?string $domain,
         ?string $context,
         string $original,
-        string $plural = null
+        ?string $plural = null
     ): ?Translation {
         $translation = parent::saveTranslation($domain, $context, $original, $plural);