Merge pull request #1323 from ToshY/rework/remove-preceding-colon-space

Removed preceding whitespace at colons in info and help messages
doctrine · Feb 16, 2023 · dc441bb · dc441bb
2 parents 7a32e47 + 9c8fe0c
commit dc441bb
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 5 deletions.
diff --git a/lib/Doctrine/Migrations/Tools/Console/Command/ExecuteCommand.php b/lib/Doctrine/Migrations/Tools/Console/Command/ExecuteCommand.php
@@ -163,7 +163,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         }
 
         $this->io->success(sprintf(
-            'Successfully migrated version(s) : %s : [%s]',
+            'Successfully migrated version(s): %s: [%s]',
             implode(', ', $versions),
             strtoupper($plan->getDirection())
         ));

diff --git a/lib/Doctrine/Migrations/Tools/Console/Command/MigrateCommand.php b/lib/Doctrine/Migrations/Tools/Console/Command/MigrateCommand.php
@@ -95,7 +95,7 @@ protected function configure(): void
 You can specify the version you wish to migrate to using an alias:
 
     <info>%command.full_name% prev</info>
-    <info>These alias are defined : first, latest, prev, current and next</info>
+    <info>These alias are defined: first, latest, prev, current and next</info>
 
 You can specify the version you wish to migrate to using an number against the current version:
 
@@ -216,7 +216,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         }
 
         $this->io->success(sprintf(
-            'Successfully migrated to version : %s',
+            'Successfully migrated to version: %s',
             $version
         ));
         $this->io->newLine();

diff --git a/tests/Doctrine/Migrations/Tests/Tools/Console/Command/ExecuteCommandTest.php b/tests/Doctrine/Migrations/Tests/Tools/Console/Command/ExecuteCommandTest.php
@@ -118,7 +118,7 @@ public function testExecute(): void
 
         self::assertSame(0, $this->executeCommandTester->getStatusCode());
         self::assertStringContainsString('[notice] Executing 1 up', trim($this->executeCommandTester->getDisplay(true)));
-        self::assertStringContainsString('[OK] Successfully migrated version(s) : 1 : [UP]', trim($this->executeCommandTester->getDisplay(true)));
+        self::assertStringContainsString('[OK] Successfully migrated version(s): 1: [UP]', trim($this->executeCommandTester->getDisplay(true)));
     }
 
     public function testExecuteMultiple(): void

diff --git a/tests/Doctrine/Migrations/Tests/Tools/Console/Command/MigrateCommandTest.php b/tests/Doctrine/Migrations/Tests/Tools/Console/Command/MigrateCommandTest.php
@@ -294,7 +294,7 @@ public function testExecuteMigrate(): void
 
         self::assertSame(0, $this->migrateCommandTester->getStatusCode());
         self::assertStringContainsString('[notice] Migrating up to A', trim($this->migrateCommandTester->getDisplay(true)));
-        self::assertStringContainsString('[OK] Successfully migrated to version : A', trim($this->migrateCommandTester->getDisplay(true)));
+        self::assertStringContainsString('[OK] Successfully migrated to version: A', trim($this->migrateCommandTester->getDisplay(true)));
     }
 
     public function testExecuteMigrateUpdatesMigrationsTableWhenNeeded(): void