Update dependencies

.github/workflows/ci.yaml

@@ -20,8 +20,25 @@ jobs:
         runs-on: ubuntu-latest
 
         strategy:
+            fail-fast: false
             matrix:
-                php-version: ["7.2", "7.3", "7.4", "8.0", "8.1"]
+                sf-version: [ '^4.4', '^5.4', '^6.3' ]
+                php-version: [ '7.2', '7.3', '7.4', '8.0', '8.1', '8.2' ]
+                exclude:
+                    -   sf-version: '^4.4'
+                        php-version: '8.0'
+                    -   sf-version: '^4.4'
+                        php-version: '8.1'
+                    -   sf-version: '^4.4'
+                        php-version: '8.2'
+                    -   sf-version: '^6.3'
+                        php-version: '7.2'
+                    -   sf-version: '^6.3'
+                        php-version: '7.3'
+                    -   sf-version: '^6.3'
+                        php-version: '7.4'
+                    -   sf-version: '^6.3'
+                        php-version: '8.0'
 
         steps:
             - uses: actions/checkout@master
@@ -45,8 +62,14 @@ jobs:
 
             - name: Install dependencies
               run: |
-                  composer install --prefer-dist --no-interaction --no-ansi --no-progress
+                  composer global config --no-plugins allow-plugins.symfony/flex true
+                  composer global require symfony/flex:^1.0
+                  export SYMFONY_REQUIRE="${{ matrix.sf-version }}"
+                  composer update --prefer-dist --no-interaction --no-ansi --no-progress
 
             - name: Run the tests suite
               run: |
                   php vendor/bin/simple-phpunit -v --log-junit ./phpunit/junit.xml
+
+            - name: Run cmd
+              run: bin/insight list

Cli/Application.php

@@ -19,6 +19,8 @@
 use SensioLabs\Insight\Sdk\Api;
 use Symfony\Component\Console\Application as SymfonyApplication;
 use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Helper\HelperSet;
+use Symfony\Component\Console\Input\InputDefinition;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Output\OutputInterface;
@@ -67,7 +69,7 @@ public function getApi()
         return $this->api;
     }
 
-    public function getLongVersion()
+    public function getLongVersion(): string
     {
         $version = parent::getLongVersion().' by <comment>Symfony</comment>';
         $commit = '@git-commit@';
@@ -79,7 +81,7 @@ public function getLongVersion()
         return $version;
     }
 
-    protected function getDefaultHelperSet()
+    protected function getDefaultHelperSet(): HelperSet
     {
         $helperSet = parent::getDefaultHelperSet();
 
@@ -89,7 +91,7 @@ protected function getDefaultHelperSet()
         return $helperSet;
     }
 
-    protected function getDefaultInputDefinition()
+    protected function getDefaultInputDefinition(): InputDefinition
     {
         $definition = parent::getDefaultInputDefinition();
 
@@ -101,7 +103,7 @@ protected function getDefaultInputDefinition()
         return $definition;
     }
 
-    protected function getDefaultCommands()
+    protected function getDefaultCommands(): array
     {
         $defaultCommands = parent::getDefaultCommands();
 
@@ -114,7 +116,7 @@ protected function getDefaultCommands()
         return $defaultCommands;
     }
 
-    protected function doRunCommand(Command $command, InputInterface $input, OutputInterface $output)
+    protected function doRunCommand(Command $command, InputInterface $input, OutputInterface $output): int
     {
         if (!$command instanceof LocalCommand\NeedConfigurationInterface) {
             return parent::doRunCommand($command, $input, $output);

Cli/Command/AnalysisCommand.php

@@ -20,7 +20,7 @@
 
 class AnalysisCommand extends Command implements NeedConfigurationInterface
 {
-    protected function configure()
+    protected function configure(): void
     {
         $this
             ->setName('analysis')
@@ -32,7 +32,7 @@ protected function configure()
         ;
     }
 
-    protected function execute(InputInterface $input, OutputInterface $output)
+    protected function execute(InputInterface $input, OutputInterface $output): int
     {
         $api = $this->getApplication()->getApi();
         $analysis = $api->getProject($input->getArgument('project-uuid'))->getLastAnalysis();

Cli/Command/AnalyzeCommand.php

@@ -22,7 +22,7 @@
 
 class AnalyzeCommand extends Command implements NeedConfigurationInterface
 {
-    protected function configure()
+    protected function configure(): void
     {
         $this
             ->setName('analyze')
@@ -38,7 +38,7 @@ protected function configure()
         ;
     }
 
-    protected function execute(InputInterface $input, OutputInterface $output)
+    protected function execute(InputInterface $input, OutputInterface $output): int
     {
         $projectUuid = $input->getArgument('project-uuid');
 

Cli/Command/ConfigureCommand.php

@@ -17,16 +17,18 @@
 
 class ConfigureCommand extends Command
 {
-    protected function configure()
+    protected function configure(): void
     {
         $this
             ->setName('configure')
             ->setDescription('(Re-)Configure your credentials.')
         ;
     }
 
-    protected function execute(InputInterface $input, OutputInterface $output)
+    protected function execute(InputInterface $input, OutputInterface $output): int
     {
         $this->getHelperSet()->get('configuration')->updateConfigurationManually($input, $output);
+
+        return 0;
     }
 }

Cli/Command/ProjectsCommand.php

@@ -18,15 +18,15 @@
 
 class ProjectsCommand extends Command implements NeedConfigurationInterface
 {
-    protected function configure()
+    protected function configure(): void
     {
         $this
             ->setName('projects')
             ->setDescription('List all your projects')
         ;
     }
 
-    protected function execute(InputInterface $input, OutputInterface $output)
+    protected function execute(InputInterface $input, OutputInterface $output): int
     {
         $api = $this->getApplication()->getApi();
 
@@ -58,5 +58,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
             ->setRows($rows)
             ->render()
         ;
+
+        return 0;
     }
 }

Cli/Command/SelfUpdateCommand.php

@@ -25,7 +25,7 @@ class SelfUpdateCommand extends Command
     /**
      * {@inheritdoc}
      */
-    protected function configure()
+    protected function configure(): void
     {
         $this
             ->setName('self-update')
@@ -44,7 +44,7 @@ protected function configure()
     /**
      * {@inheritdoc}
      */
-    protected function execute(InputInterface $input, OutputInterface $output)
+    protected function execute(InputInterface $input, OutputInterface $output): int
     {
         $remoteFilename = 'http://get.insight.sensiolabs.com/insight.phar';
         $localFilename = $_SERVER['argv'][0];
@@ -57,7 +57,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
                 $output->writeln('<info>insight is already up to date.</info>');
                 unlink($tempFilename);
 
-                return;
+                return 0;
             }
 
             chmod($tempFilename, 0777 & ~umask());
@@ -69,13 +69,17 @@ protected function execute(InputInterface $input, OutputInterface $output)
             rename($tempFilename, $localFilename);
 
             $output->writeln('<info>insight updated.</info>');
+
+            return 0;
         } catch (\Exception $e) {
             if (!$e instanceof \UnexpectedValueException && !$e instanceof \PharException) {
                 throw $e;
             }
             unlink($tempFilename);
             $output->writeln('<error>The download is corrupt ('.$e->getMessage().').</error>');
             $output->writeln('<error>Please re-run the self-update command to try again.</error>');
+
+            return 1;
         }
     }
 }

Cli/Helper/ConfigurationHelper.php

@@ -62,7 +62,7 @@ public function getConfiguration(InputInterface $input, OutputInterface $output)
         return $configuration;
     }
 
-    public function getName()
+    public function getName(): string
     {
         return 'configuration';
     }

Cli/Helper/DescriptorHelper.php

@@ -61,7 +61,7 @@ public function register($format, AbstractDescriptor $descriptor)
     /**
      * {@inheritdoc}
      */
-    public function getName()
+    public function getName(): string
     {
         return 'descriptor';
     }

Cli/Helper/FailConditionHelper.php

@@ -69,7 +69,7 @@ public function evaluate(Analysis $analysis, $expr)
         return 0;
     }
 
-    public function getName()
+    public function getName(): string
     {
         return 'fail_condition';
     }

composer.json

@@ -7,14 +7,14 @@
         "ext-json": "*",
         "jms/serializer": "^3.17",
         "psr/log": "^1.0",
-        "symfony/http-client": "^4.4",
-        "symfony/console": "^4.4",
-        "symfony/expression-language": "^4.4"
+        "symfony/http-client": "^4.4|^5.4|^6.3",
+        "symfony/console": "^4.4|^5.4|^6.3",
+        "symfony/expression-language": "^4.4|^5.4|^6.3"
     },
     "require-dev": {
         "monolog/monolog": "^1.4",
-        "symfony/phpunit-bridge": "^4.4|^5.3",
-        "symfony/var-dumper": "^4.4|^5.3"
+        "symfony/phpunit-bridge": "^4.4|^5.4|^6.3",
+        "symfony/var-dumper": "^4.4|^5.4|^6.3"
     },
     "suggest": {
         "monolog/monolog": "Will add some log capability to this library"
@@ -25,14 +25,9 @@
         }
     },
     "bin": ["bin/insight"],
-    "config": {
-        "platform": {
-            "php": "7.2.5"
-        }
-    },
     "extra": {
         "branch-alias": {
-            "dev-master": "1.5.x-dev"
+            "dev-master": "1.10.x-dev"
         }
     }
 }