Merge pull request #52 from elifesciences/update-symfony-console-sign…

…atures

Widen support for symfony console 5.x

composer.json

@@ -25,7 +25,7 @@
     "require-dev": {
         "phpunit/phpunit": "^6.5 | ^9.5",
         "squizlabs/php_codesniffer": "^3.5",
-        "symfony/console": "^2.7 || ^3.2"
+        "symfony/console": "^2.7 || ^3.2 || ^4.4 || ^5.0"
     },
     "conflict": {
         "sebastian/comparator": "<1.2.3"

src/Command/QueueCleanCommand.php

@@ -28,9 +28,11 @@ protected function configure()
             ->setDescription('Cleans the SQS queue through purging.');
     }
 
-    public function execute(InputInterface $input, OutputInterface $output)
+    public function execute(InputInterface $input, OutputInterface $output): int
     {
         $this->logger->info('Cleaning queue: '.$this->queue->getName());
         $this->queue->clean();
+
+        return 0;
     }
 }

src/Command/QueueCommand.php

@@ -56,13 +56,15 @@ protected function configure()
             ->setDescription('Watches SQS for changes.');
     }
 
-    final public function execute(InputInterface $input, OutputInterface $output)
+    final public function execute(InputInterface $input, OutputInterface $output): int
     {
         $this->logger->info($this->getName().' Started listening.');
         while (!$this->limit->hasBeenReached()) {
             $this->loop($input);
         }
         $this->logger->info($this->getName().' Stopped because of limits reached.');
+
+        return 0;
     }
 
     final protected function transform(QueueItem $item)

src/Command/QueueCountCommand.php

@@ -25,8 +25,10 @@ protected function configure()
             ->setDescription('Counts the SQS messages, including invisible ones. Approximate.');
     }
 
-    public function execute(InputInterface $input, OutputInterface $output)
+    public function execute(InputInterface $input, OutputInterface $output): int
     {
         $output->writeln($this->queue->count());
+
+        return 0;
     }
 }