fixup! Add ECS

Author: BenMorel

.github/workflows/coding-standard.yml

@@ -8,3 +8,5 @@ jobs:
   coding-standard:
     name: Coding Standard
     uses: brick/coding-standard/.github/workflows/coding-standard.yml@main
+    with:
+        php-version: "8.2"

tools/ecs/ecs.php

@@ -3,8 +3,8 @@
 declare(strict_types=1);
 
 use PHP_CodeSniffer\Standards\Squiz\Sniffs\Commenting\FunctionCommentSniff;
-use PhpCsFixer\Fixer\Whitespace\MethodChainingIndentationFixer;
-use SlevomatCodingStandard\Sniffs\Namespaces\ReferenceUsedNamesOnlySniff;
+use PhpCsFixer\Fixer\ClassNotation\OrderedTypesFixer;
+use PhpCsFixer\Fixer\Phpdoc\PhpdocTypesOrderFixer;
 use SlevomatCodingStandard\Sniffs\Namespaces\UseSpacingSniff;
 use SlevomatCodingStandard\Sniffs\Whitespaces\DuplicateSpacesSniff;
 use Symplify\EasyCodingStandard\Config\ECSConfig;
@@ -26,13 +26,6 @@
 
     // PHP-CS-Fixer
     $ecsConfig->skip([
-        // Allows tree building method chaining syntax for better readability
-        MethodChainingIndentationFixer::class => [$libRootPath . '/src/Parser/IsoParsers.php'],
-
-        // Allows microtime() to be called from class namespace so that it can be overridden
-        // and its return value mocked in SystemClockTest
-        ReferenceUsedNamesOnlySniff::class => [$libRootPath . '/src/Clock/SystemClock.php'],
-
         // Only interested in FunctionCommentSniff.ParamCommentFullStop, excludes the rest
         FunctionCommentSniff::class . '.Missing' => null,
         FunctionCommentSniff::class . '.MissingReturn' => null,
@@ -50,5 +43,8 @@
         // Keep a line between same use types, spacing around uses is done in other fixers
         UseSpacingSniff::class . '.IncorrectLinesCountBeforeFirstUse' => null,
         UseSpacingSniff::class . '.IncorrectLinesCountAfterLastUse' => null,
+
+        OrderedTypesFixer::class => null,
+        PhpdocTypesOrderFixer::class => null,
     ]);
 };