fix: Explicitly mark parameters as nullable

phpstan.neon.dist

@@ -3,13 +3,13 @@ parameters:
     paths:
         - src
         - tests
-    checkMissingIterableValueType: false
-    checkGenericClassInNonGenericObjectType: false
     earlyTerminatingMethodCalls:
         Version\Comparison\Constraint\OperationConstraintParser:
             - error
     ignoreErrors:
             - '~Method [a-zA-Z0-9\\_]+::matches\(\) has parameter \$[a-zA-Z0-9]+ with no type specified.~'
+            - identifier: missingType.generics
+            - identifier: missingType.iterableValue
 
 
 includes:

src/Version.php

@@ -31,7 +31,7 @@ class Version implements JsonSerializable
 
     protected static ?Comparator $comparator = null;
 
-    final protected function __construct(int $major, int $minor, int $patch, PreRelease $preRelease = null, Build $build = null)
+    final protected function __construct(int $major, int $minor, int $patch, ?PreRelease $preRelease = null, ?Build $build = null)
     {
         VersionAssert::that($major)->greaterOrEqualThan(0, 'Major version must be positive integer');
         VersionAssert::that($minor)->greaterOrEqualThan(0, 'Minor version must be positive integer');
@@ -44,7 +44,7 @@ final protected function __construct(int $major, int $minor, int $patch, PreRele
         $this->build = $build;
     }
 
-    public static function from(int $major, int $minor = 0, int $patch = 0, PreRelease $preRelease = null, Build $build = null): Version
+    public static function from(int $major, int $minor = 0, int $patch = 0, ?PreRelease $preRelease = null, ?Build $build = null): Version
     {
         return new static($major, $minor, $patch, $preRelease, $build);
     }
@@ -59,13 +59,13 @@ public static function fromString(string $versionString): Version
         }
 
         $version = new static(
-            (int) $parts['major'],
-            (int) $parts['minor'],
-            (int) $parts['patch'],
+            (int)$parts['major'],
+            (int)$parts['minor'],
+            (int)$parts['patch'],
             (isset($parts['preRelease']) && '' !== $parts['preRelease']) ? PreRelease::fromString($parts['preRelease']) : null,
             (isset($parts['build']) && '' !== $parts['build']) ? Build::fromString($parts['build']) : null
         );
-        $version->prefix = $parts['prefix'] ?? '';
+        $version->prefix = $parts['prefix'];
 
         return $version;
     }
@@ -208,8 +208,7 @@ public function toString(): string
             . '.' . $this->minor
             . '.' . $this->patch
             . (($this->preRelease !== null) ? '-' . $this->preRelease->toString() : '')
-            . (($this->build !== null) ? '+' . $this->build->toString() : '')
-        ;
+            . (($this->build !== null) ? '+' . $this->build->toString() : '');
     }
 
     public function __toString(): string

tests/TestAsset/VersionIsIdentical.php

@@ -14,11 +14,11 @@ final class VersionIsIdentical extends Constraint
     private Version $expectedVersion;
 
     public function __construct(
-        int $expectedMajor,
-        int $expectedMinor,
-        int $expectedPatch,
-        PreRelease|string $expectedPreRelease = null,
-        Build|string $expectedBuild = null
+        int                    $expectedMajor,
+        int                    $expectedMinor,
+        int                    $expectedPatch,
+        PreRelease|string|null $expectedPreRelease = null,
+        Build|string|null      $expectedBuild = null
     ) {
         $this->expectedVersion = Version::from(
             $expectedMajor,