style: fix coding styles (#475)

.php-cs-fixer.dist.php

@@ -42,5 +42,5 @@
 return Factory::create(new CodeIgniter4(), $overrides, $options)->forLibrary(
     'CodeIgniter 4 framework',
     'CodeIgniter Foundation',
-    '[email protected]'
+    '[email protected]',
 );

bin/generate-changelog

@@ -21,7 +21,7 @@ if (! isset($argv[1]) || ! in_array($argv[1], ['--major', '--minor', '--patch'],
     CLI::error(
         'Please provide the tagging strategy for this release: --major, --minor, --patch.',
         'light_gray',
-        'red'
+        'red',
     );
 
     exit(1);
@@ -74,7 +74,7 @@ $nextTag = preg_replace_callback(
                 return sprintf('v%d.0.0', ++$matches[1]);
         }
     },
-    $latestTag
+    $latestTag,
 );
 
 CLI::write(sprintf('Generating changelog for %s...', $nextTag), 'green');
@@ -101,9 +101,9 @@ foreach ($changelog as $log) {
                 static fn (array $matches): string => preg_replace(
                     '/(?<!^)((?=[\p{Lu}][^\p{Lu}])|(?<![\p{Lu}])(?=[\p{Lu}]))/u',
                     ' ',
-                    $matches[1]
+                    $matches[1],
                 ),
-                $class
+                $class,
             );
 
             continue;
@@ -135,9 +135,9 @@ CLI::write(sprintf(
     implode("\n", array_map(
         static fn (string $value, string $key): string => sprintf('- %s (%s)', $key, $value),
         array_values($localeChanges),
-        array_keys($localeChanges)
+        array_keys($localeChanges),
     )) ?: '- <none>',
-    implode("\n- ", $otherChanges) ?: '<none>'
+    implode("\n- ", $otherChanges) ?: '<none>',
 ));
 
 sleep(1); // for cinematic effect

bin/generate-page

@@ -59,7 +59,7 @@ $enFiles = get_filenames(
     $enDir,
     true,
     false,
-    false
+    false,
 );
 
 $enItems = [];
@@ -112,7 +112,7 @@ foreach ($locales as $localeKey => $locale) {
         $langDir,
         true,
         false,
-        false
+        false,
     );
 
     $items[$locale] = [
@@ -121,9 +121,9 @@ foreach ($locales as $localeKey => $locale) {
             static fn (array $matches): string => preg_replace(
                 '/(?<!^)((?=[\p{Lu}][^\p{Lu}])|(?<![\p{Lu}])(?=[\p{Lu}]))/u',
                 ' ',
-                $matches[1]
+                $matches[1],
             ),
-            $localeKey
+            $localeKey,
         ),
         'locale'     => $locale,
         'percent'    => 0,

bin/test

@@ -66,7 +66,7 @@ if ($argc === 2) {
 CLI::write(
     'The following locales have untranslated items:',
     'light_gray',
-    'red'
+    'red',
 );
 exec($phpunit, $output, $resultCode);
 

bin/update-en-comments

@@ -46,14 +46,14 @@ $files = get_filenames(
     $langDir,
     true,
     false,
-    false
+    false,
 );
 
 $enFiles = get_filenames(
     $enDir,
     true,
     false,
-    false
+    false,
 );
 
 foreach ($enFiles as $enFile) {

tests/AutoReview/TranslationsCodeTest.php

@@ -41,7 +41,7 @@ public function testAllTranslatedLocalesAreIncludedInTheReadMe(): void
         foreach ($this->locales() as $locale) {
             $this->assertSame(1, preg_match(sprintf('/\| %s \s+\|/', $locale), $readme), sprintf(
                 'The locale "%s" is not found in the README.md file. Please add it.',
-                $locale
+                $locale,
             ));
         }
     }
@@ -53,7 +53,7 @@ public function testAllTranslatedLocalesAreNotIncludedInMissing(): void
         foreach ($this->locales() as $locale) {
             $this->assertSame(0, preg_match(sprintf('/\| %s \s+\|/', $locale), $missing), sprintf(
                 'The locale "%s" is already translated and should no longer be found in MISSING.md file.',
-                $locale
+                $locale,
             ));
         }
     }
@@ -79,7 +79,7 @@ private function getContentsOrFail(string $filepath): string
         if ($contents === false) {
             $this->fail(sprintf(
                 'Failed to get the contents of %s. It is either not found or not readable.',
-                basename($filepath)
+                basename($filepath),
             ));
         }
 

tests/Language/AbstractTranslationTestCase.php

@@ -102,7 +102,7 @@ final public function testAllConfiguredLanguageFilesAreTranslated(string $locale
     {
         $filesNotTranslated = array_diff(
             $this->expectedSets(),
-            $this->foundSets($locale)
+            $this->foundSets($locale),
         );
 
         sort($filesNotTranslated);
@@ -113,7 +113,7 @@ final public function testAllConfiguredLanguageFilesAreTranslated(string $locale
             $count > 1 ? 'files' : 'file',
             implode('", "', $filesNotTranslated),
             $count > 1 ? 'are' : 'is',
-            $locale
+            $locale,
         ));
     }
 
@@ -126,7 +126,7 @@ final public function testAllTranslatedLanguageFilesAreConfigured(string $locale
     {
         $filesNotConfigured = array_diff(
             $this->foundSets($locale),
-            $this->expectedSets()
+            $this->expectedSets(),
         );
 
         sort($filesNotConfigured);
@@ -137,7 +137,7 @@ final public function testAllTranslatedLanguageFilesAreConfigured(string $locale
             $count > 1 ? 'files' : 'file',
             implode('", "', $filesNotConfigured),
             $locale,
-            $count > 1 ? 'are' : 'is'
+            $count > 1 ? 'are' : 'is',
         ));
     }
 
@@ -153,7 +153,7 @@ final public function testAllConfiguredLanguageKeysAreIncluded(string $locale):
         foreach ($this->foundSets($locale) as $file) {
             $missing = array_diff_key(
                 $this->loadFile($file),
-                $this->loadFile($file, $locale)
+                $this->loadFile($file, $locale),
             );
 
             foreach (array_keys($missing) as $key) {
@@ -169,7 +169,7 @@ final public function testAllConfiguredLanguageKeysAreIncluded(string $locale):
             $count > 1 ? 'keys' : 'key',
             implode('", "', $keysNotIncluded),
             $count > 1 ? 'are' : 'is',
-            $locale
+            $locale,
         ));
     }
 
@@ -185,7 +185,7 @@ final public function testAllIncludedLanguageKeysAreConfigured(string $locale):
         foreach ($this->foundSets($locale) as $file) {
             $extra = array_diff_key(
                 $this->loadFile($file, $locale),
-                $this->loadFile($file)
+                $this->loadFile($file),
             );
 
             foreach (array_keys($extra) as $key) {
@@ -201,7 +201,7 @@ final public function testAllIncludedLanguageKeysAreConfigured(string $locale):
             $count > 1 ? 'keys' : 'key',
             implode('", "', $keysNotConfigured),
             $locale,
-            $count > 1 ? 'are' : 'is'
+            $count > 1 ? 'are' : 'is',
         ));
     }
 
@@ -253,7 +253,7 @@ final public function testAllIncludedLanguageKeysAreTranslated(string $locale):
             $count > 1 ? 'keys' : 'key',
             implode('", "', $keysNotTranslated),
             $locale,
-            $count > 1 ? 'differ' : 'differs'
+            $count > 1 ? 'differ' : 'differs',
         ));
     }
 
@@ -283,7 +283,7 @@ final public function testAllConfiguredLanguageKeysAreInOrder(string $locale): v
                             "\n%s:\n%s\n%s",
                             $file,
                             CLI::color("-'{$expectedKey}' => '{$original[$expectedKey]}';", 'red'),
-                            CLI::color("+'{$actualKey}' => '{$translated[$actualKey]}';", 'green')
+                            CLI::color("+'{$actualKey}' => '{$translated[$actualKey]}';", 'green'),
                         );
                         break;
                     }
@@ -295,7 +295,7 @@ final public function testAllConfiguredLanguageKeysAreInOrder(string $locale): v
             "Failed asserting that the translated language keys in \"%s\" locale are ordered correctly.\n%s\n%s",
             $locale,
             CLI::color('--- Original', 'red') . "\n" . CLI::color('+++ Translated', 'green'),
-            implode("\n", $diffs)
+            implode("\n", $diffs),
         ));
     }
 
@@ -320,7 +320,7 @@ final public function testLocaleHasCorrespondingTestCaseFile(string $locale): vo
 
         $this->assertTrue(class_exists($class, false), sprintf(
             'Failed asserting that test class "%s" is existing.',
-            $class
+            $class,
         ));
     }