Skip to content

Add int return type to NodesValidatorCommand::execute() - #412

Open
itaymesh wants to merge 2 commits into
SnowdogApps:developfrom
studioraz:copilot/fix-execute-return-type
Open

Add int return type to NodesValidatorCommand::execute()#412
itaymesh wants to merge 2 commits into
SnowdogApps:developfrom
studioraz:copilot/fix-execute-return-type

Conversation

@itaymesh

@itaymesh itaymesh commented Jul 28, 2026

Copy link
Copy Markdown

Symfony Console's Command::execute() declares an int return type, but the override in NodesValidatorCommand lacked the type hint, causing a signature mismatch.

Change

  • Console/Command/NodesValidatorCommand.php: add : int return type to execute()
-    public function execute(InputInterface $input, OutputInterface $output)
+    public function execute(InputInterface $input, OutputInterface $output): int

The method body already returns 0 in all paths; no logic changes were needed.

@itaymesh itaymesh changed the title Copilot/fix execute return type Add int return type to NodesValidatorCommand::execute() Jul 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants