fixup! cs

Author: TomasVotruba

src/Command/RaiseToInstalledCommand.php

@@ -10,6 +10,7 @@
 use Nette\Utils\Json;
 use Rector\Jack\FileSystem\ComposerJsonPackageVersionUpdater;
 use Rector\Jack\Utils\JsonFileLoader;
+use Rector\Jack\ValueObject\ChangedPackageVersion;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Input\InputOption;
@@ -95,7 +96,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
                         $newPackageVersion
                     );
 
-                    $changedPackages[] = $packageName;
+                    $changedPackages[] = new ChangedPackageVersion($packageName, $packageVersion, $newPackageVersion);
                     continue;
                 }
             }
@@ -111,8 +112,9 @@ protected function execute(InputInterface $input, OutputInterface $output): int
             // remove "-dev" suffix
             $normalizedConstraintVersion = str_replace('-dev', '', $normalizedConstraintVersion);
 
-            // all equal
-            if ($normalizedConstraintVersion === $normalizedInstalledVersion) {
+
+            // are major + minor equal?
+            if ($this->areMajorAndMinorVersionsEqual($normalizedConstraintVersion, $normalizedInstalledVersion)) {
                 continue;
             }
 
@@ -129,7 +131,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
 
             // focus on minor only
             // or on patch in case of 0.*
-            $changedPackages[] = $packageName;
+            $changedPackages[] = new ChangedPackageVersion($packageName, $packageVersion, $newRequiredVersion);
         }
 
         if ($changedPackages === []) {
@@ -142,13 +144,25 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         }
 
         $symfonyStyle->success(sprintf(
-            '%d packages %s changed to installed versions.%s%s "composer update --lock" to update "composer.lock" hash',
+            '%d package%s %s changed to installed versions.%s%s "composer update --lock" to update "composer.lock" hash',
             count($changedPackages),
+            count($changedPackages) === 1 ? '' : 's',
             $isDryRun ? 'would be (is "--dry-run")' : 'were updated',
+            PHP_EOL,
             $isDryRun ? 'Then you would run' : 'Now run',
-            PHP_EOL
         ));
 
+        foreach ($changedPackages as $changedPackage) {
+            $symfonyStyle->writeln(sprintf(
+                ' * <fg=green>%s</> (<fg=yellow>%s</> => <fg=yellow>%s</>)',
+                $changedPackage->getPackageName(),
+                $changedPackage->getOldVersion(),
+                $changedPackage->getNewVersion()
+            ));
+        }
+
+        $symfonyStyle->newLine();
+
         return self::SUCCESS;
     }
 
@@ -172,4 +186,13 @@ private function resolveInstalledPackagesToVersions(): array
 
         return $installedPackagesToVersions;
     }
+
+    private function areMajorAndMinorVersionsEqual(string $firstVersion, string $secondVersion): bool
+    {
+        [$firstMajor, $firstMinor] = explode('.', $firstVersion);
+        [$secondMajor, $secondMinor] = explode('.', $secondVersion);
+
+        // if major and minor are equal, we can skip the update
+        return $firstMajor === $secondMajor && $firstMinor === $secondMinor;
+    }
 }

src/ValueObject/ChangedPackageVersion.php

@@ -0,0 +1,32 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\Jack\ValueObject;
+
+final class ChangedPackageVersion
+{
+    public function __construct(
+        private string $packageName,
+        private string $oldVersion,
+        private string $newVersion,
+    )
+    {
+
+    }
+
+    public function getPackageName(): string
+    {
+        return $this->packageName;
+    }
+
+    public function getOldVersion(): string
+    {
+        return $this->oldVersion;
+    }
+
+    public function getNewVersion(): string
+    {
+        return $this->newVersion;
+    }
+}