Merge pull request #102 from nickvergessen/bugfix/noid/php-8.4-compatibility

fix: Make $context explicitly nullable

Author: stecman

src/CompletionHandler.php

@@ -40,7 +40,7 @@ class CompletionHandler
      */
     private $commandWordIndex;
 
-    public function __construct(Application $application, CompletionContext $context = null)
+    public function __construct(Application $application, ?CompletionContext $context = null)
     {
         $this->application = $application;
         $this->context = $context;

src/HookFactory.php

@@ -126,10 +126,10 @@ public static function getShellTypes()
     /**
      * Return a completion hook for the specified shell type
      *
-     * @param string $type - a key from self::$hooks
-     * @param string $programPath
-     * @param string $programName
-     * @param bool   $multiple
+     * @param string  $type - a key from self::$hooks
+     * @param string  $programPath
+     * @param ?string $programName
+     * @param bool    $multiple
      *
      * @return string
      */

tests/Stecman/Component/Symfony/Console/BashCompletion/Common/CompletionHandlerTestCase.php

@@ -47,7 +47,7 @@ protected function setUp(): void
      * Create a handler set up with the given commandline and cursor position
      *
      * @param $commandLine
-     * @param int $cursorIndex
+     * @param ?int $cursorIndex
      * @return CompletionHandler
      */
     protected function createHandler($commandLine, $cursorIndex = null)