Merge pull request #94 from rajeshreeputra/rajeshreeputra-patch-1

stecman · web-flow · commit 90a1d2c3d51b · 2023-06-24T10:17:30.000+12:00
Update CompletionCommand.php to Add return type (broken with console &gt;=6.3)
diff --git a/src/CompletionCommand.php b/src/CompletionCommand.php
@@ -51,7 +51,7 @@ public function getNativeDefinition(): InputDefinition
      * Any global options defined by user-code are meaningless to this command.
      * Options outside of the core defaults are ignored to avoid name and shortcut conflicts.
      */
-    public function mergeApplicationDefinition($mergeArgs = true)
+    public function mergeApplicationDefinition($mergeArgs = true): void
     {
         // Get current application options
         $appDefinition = $this->getApplication()->getDefinition();

Original file line number	Diff line number	Diff line change
`@@ -51,7 +51,7 @@ public function getNativeDefinition(): InputDefinition`
`51`	`51`	`* Any global options defined by user-code are meaningless to this command.`
`52`	`52`	`* Options outside of the core defaults are ignored to avoid name and shortcut conflicts.`
`53`	`53`	`*/`
`54`		`- public function mergeApplicationDefinition($mergeArgs = true)`
	`54`	`+ public function mergeApplicationDefinition($mergeArgs = true): void`
`55`	`55`	`{`
`56`	`56`	`// Get current application options`
`57`	`57`	`$appDefinition = $this->getApplication()->getDefinition();`