Update ECS

Author: chrisminett

composer.json

@@ -29,20 +29,17 @@
     "provide": {
         "psr/log-implementation": "1.0 || 2.0 || 3.0"
     },
-
     "require-dev": {
         "phpunit/phpunit": "^9",
         "symplify/easy-coding-standard": "^12"
     },
-
     "autoload-dev": {
         "psr-4": {
             "Phlib\\Logger\\Test\\": "tests"
         }
     },
-
     "scripts": {
-        "cs-check": "phpcs --standard=PSR12 src/ tests/",
-        "cs-fix": "phpcbf --standard=PSR12 src/ tests/"
+        "check-cs": "vendor/bin/ecs check --ansi",
+        "fix-cs": "vendor/bin/ecs check --fix --ansi"
     }
 }

ecs.php

@@ -10,6 +10,7 @@
         __DIR__ . '/src',
         __DIR__ . '/tests',
     ])
+    ->withRootFiles()
 
     ->withSets([
         SetList::COMMON,
@@ -23,13 +24,28 @@
 
         // Remove sniff, from common/spaces
         \PhpCsFixer\Fixer\Operator\NotOperatorWithSuccessorSpaceFixer::class,
+    ])
+
+    // Rule from common/spaces. No space after cast.
+    ->withConfiguredRule(
         \PhpCsFixer\Fixer\CastNotation\CastSpacesFixer::class,
+        [
+            'space' => 'none',
+        ],
+    )
+
+    /*
+     * Rule missing from PSR12. PER Coding Style 3:
+     * "... each of the blocks [of import statements] MUST be separated by a single blank line ..."
+     */
+    ->withRules([
+        \PhpCsFixer\Fixer\Whitespace\BlankLineBetweenImportGroupsFixer::class,
     ])
 
-    // PER Coding Style 7.1: "The `fn` keyword MUST NOT be succeeded by a space."
+    // Rule from PSR12. PER Coding Style 7.1: "The `fn` keyword MUST NOT be succeeded by a space."
     ->withConfiguredRule(
         \PhpCsFixer\Fixer\FunctionNotation\FunctionDeclarationFixer::class,
         [
             'closure_fn_spacing' => 'none',
-        ]
+        ],
     );

src/Config.php

@@ -5,7 +5,6 @@
 namespace Phlib\Logger;
 
 /**
- * Class Config
  * @package Phlib\Logger
  */
 class Config implements ConfigInterface

src/ConfigInterface.php

@@ -5,7 +5,6 @@
 namespace Phlib\Logger;
 
 /**
- * Interface ConfigInterface
  * @package Phlib\Logger
  */
 interface ConfigInterface

src/Decorator/AbstractDecorator.php

@@ -18,8 +18,6 @@ abstract class AbstractDecorator extends AbstractLogger
     private $logger;
 
     /**
-     * AbstractDecorator constructor
-     *
      * Stores the Logger for re-use in the concrete via getInnerLogger()
      *
      * If the concrete requires use of the config value, override the constructor

src/Decorator/LevelFilter.php

@@ -9,7 +9,6 @@
 use Psr\Log\LogLevel;
 
 /**
- * Class LevelFilter
  * @package Phlib\Logger
  */
 class LevelFilter extends AbstractDecorator

src/Factory.php

@@ -7,7 +7,6 @@
 use Psr\Log\LoggerInterface;
 
 /**
- * Class Factory
  * @package Phlib\Logger
  */
 class Factory

src/LoggerType/CliColor.php

@@ -8,8 +8,6 @@
 use Symfony\Component\Console\Formatter\OutputFormatterStyle;
 
 /**
- * Class CliColor
- *
  * Write to CLI with colour!
  *
  * @package Phlib\Logger

src/LoggerType/Collection.php

@@ -8,7 +8,6 @@
 use Psr\Log\LoggerInterface;
 
 /**
- * Class Collection
  * @package Phlib\Logger
  */
 class Collection extends AbstractLogger

src/LoggerType/Stream.php

@@ -7,7 +7,6 @@
 use Psr\Log\AbstractLogger;
 
 /**
- * Class Stream
  * @package Phlib\Logger
  */
 class Stream extends AbstractLogger