Bump console version (#7)

* Bump console version
* Fix coding standards

Author: kimpepper

.github/workflows/build.yml

@@ -17,7 +17,7 @@ jobs:
       matrix:
         prefer_lowest: ["", "--prefer-lowest"]
     container:
-      image: skpr/php-cli:8.2-dev-v2-edge
+      image: skpr/php-cli:8.3-dev-v2-latest
       options:
         --pull always
         --user 1001:1001

composer.json

@@ -13,13 +13,13 @@
   "require": {
     "php": "^8.1",
     "ext-simplexml": "*",
-    "symfony/console": "^6.3"
+    "symfony/console": "^6.3|^7.0"
   },
   "require-dev": {
     "dealerdirect/phpcodesniffer-composer-installer": "^1.0",
     "phpstan/phpstan": "^1.10",
     "phpunit/phpunit": "^9.6",
-    "previousnext/coding-standard": "^0.1.3",
+    "previousnext/coding-standard": "^1.1.1",
     "staabm/annotate-pull-request-from-checkstyle": "^1.8"
   },
   "bin" : ["phpunit-splitter"],

phpcs.xml

@@ -3,6 +3,8 @@
   <file>./src</file>
   <file>./tests/src</file>
   <rule ref="PreviousNextDrupal" />
-  <rule ref="SlevomatCodingStandard.Namespaces.FullyQualifiedGlobalFunctions" />
+  <rule ref="SlevomatCodingStandard.Commenting.UselessInheritDocComment" >
+    <severity>0</severity>
+  </rule>
   <arg name="report" value="full"/>
 </ruleset>

src/GlobbingTestResultCache.php

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace PhpUnitSplitter;
 
@@ -34,7 +34,9 @@ final class GlobbingTestResultCache {
   /**
    * Constructs a new GlobbingTestResultCache.
    */
-  public function __construct(string $filepaths = ".phpunit.cache/test-results*") {
+  public function __construct(
+    string $filepaths = ".phpunit.cache/test-results*",
+  ) {
     $filenames = \glob($filepaths);
     if ($filenames === FALSE) {
       $filenames = [];

src/SplitterCommand.php

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace PhpUnitSplitter;
 
@@ -71,7 +71,7 @@ protected function configure(): void {
    * {@inheritdoc}
    */
   protected function execute(InputInterface $input, OutputInterface $output): int {
-    \set_error_handler(fn($severity, $message, $file, $line) => throw new \ErrorException($message, 0, $severity, $file, $line));
+    \set_error_handler(static fn($severity, $message, $file, $line) => throw new \ErrorException($message, 0, $severity, $file, $line));
     $bootstrap = $input->getOption('bootstrap-file');
     if (\file_exists($bootstrap)) {
       include_once $bootstrap;

src/TestMapper.php

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace PhpUnitSplitter;
 
@@ -18,7 +18,11 @@ final class TestMapper {
   /**
    * Constructs a new TestMapper.
    */
-  public function __construct(string $testListFilePath, string $testResultFiles, string $prefix) {
+  public function __construct(
+    string $testListFilePath,
+    string $testResultFiles,
+    string $prefix,
+  ) {
     $this->testsXml = \simplexml_load_file($testListFilePath);
     $this->resultCache = new GlobbingTestResultCache($testResultFiles);
     $this->prefix = $prefix;
@@ -82,7 +86,7 @@ public function getMap(): array {
    *   The sorted map.
    */
   public function sortMap(array $map): array {
-    \uasort($map, function ($a, $b) {
+    \uasort($map, static function ($a, $b) {
       return $a['time'] <=> $b['time'];
     });
     return $map;

src/TestStatus.php

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace PhpUnitSplitter;
 

tests/src/GlobbingTestResultCacheTest.php

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace PhpUnitSplitter\Tests;
 

tests/src/PhpUnitSplitterTest.php

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace PhpUnitSplitter\Tests;