PHP linting

Author: jigarius

src/Argument/Argument.php

@@ -19,7 +19,7 @@ protected function __construct(
     string $name,
     string $description = '',
     bool $isOptional = TRUE,
-    $defaultValue = NULL
+    $defaultValue = NULL,
   ) {
     $this->name = $name;
     $this->description = $description;
@@ -31,7 +31,7 @@ final public static function create(
     string $name,
     string $description = '',
     bool $isOptional = TRUE,
-    string|int|float|bool|NULL $defaultValue = NULL
+    string|int|float|bool|NULL $defaultValue = NULL,
   ): ArgumentInterface {
     return match ($name) {
       'command', 'input', 'output' => new BuiltInArgument($name),

src/Argument/BuiltInArgument.php

@@ -9,11 +9,11 @@ class BuiltInArgument extends Argument implements ArgumentInterface {
 
   protected function __construct(
     string $name,
-    string $description = NULL,
-    bool $isOptional = NULL,
-    $defaultValue = NULL
+    string $description = '',
+    bool $isOptional = TRUE,
+    $defaultValue = NULL,
   ) {
-    parent::__construct($name, '', TRUE, NULL);
+    parent::__construct($name);
   }
 
   public function getDescription(): string {

src/Argument/VariadicArgument.php

@@ -11,7 +11,7 @@ protected function __construct(
     string $name,
     string $description,
     bool $isOptional,
-    $defaultValue
+    $defaultValue,
   ) {
     if (!is_null($defaultValue)) {
       throw new \InvalidArgumentException('Variadic argument default value must be NULL');

src/Phpake.php

@@ -63,7 +63,7 @@ public function require(string $path) {
   /**
    * {@inheritdoc}
    */
-  public function run(InputInterface $input = NULL, OutputInterface $output = NULL): int {
+  public function run(?InputInterface $input = NULL, ?OutputInterface $output = NULL): int {
     return parent::run($input ?? $this->input, $output ?? $this->output);
   }