Merge pull request #91 from lmc-eu/feature/internal-ecs-config

Style: Add internal rules configuration for the project itself

Author: OndraM

.gitattributes

@@ -0,0 +1 @@
+ecs-internal.php export-ignore

composer.json

@@ -55,12 +55,12 @@
             "@test"
         ],
         "analyze": [
-            "vendor/bin/ecs check src/ tests/ ecs.php ecs-8.1.php ecs-8.2.php ecs-8.3.php --ansi",
+            "vendor/bin/ecs check --config=ecs-internal.php --ansi",
             "vendor/bin/phpstan analyze -c phpstan.neon --ansi"
         ],
         "fix": [
             "@composer normalize",
-            "vendor/bin/ecs check ./src/ ./tests/ ecs.php ecs-8.1.php ecs-8.2.php ecs-8.3.php --ansi --fix"
+            "vendor/bin/ecs check --config=ecs-internal.php --ansi --fix"
         ],
         "lint": [
             "vendor/bin/parallel-lint -j 10 -e php ./src ./tests ecs.php ecs-8.1.php ecs-8.2.php ecs-8.3.php",

ecs-internal.php

@@ -0,0 +1,22 @@
+<?php declare(strict_types=1);
+
+use PhpCsFixer\Fixer\PhpUnit\PhpUnitTestAnnotationFixer;
+use Symplify\CodingStandard\Fixer\LineLength\LineLengthFixer;
+use Symplify\EasyCodingStandard\Config\ECSConfig;
+
+/**
+ * Internal rules configuration for the lmc/coding-standard project itself
+ */
+return ECSConfig::configure()
+    ->withPaths([__DIR__ . '/src', __DIR__ . '/tests'])
+    ->withRootFiles()
+    ->withSets(
+        [
+            __DIR__ . '/ecs.php',
+        ],
+    )
+    ->withConfiguredRule(PhpUnitTestAnnotationFixer::class, ['style' => 'annotation'])
+    ->withConfiguredRule(
+        LineLengthFixer::class,
+        ['line_length' => 120, 'break_long_lines' => true, 'inline_short_lines' => false],
+    );

ecs.php

@@ -461,7 +461,10 @@
     ->withConfiguredRule(ArraySyntaxFixer::class, ['syntax' => 'short'])
     // The body of each structure MUST be enclosed by braces. Braces should be properly placed
     // @TODO move configuration to BracesPositionFixer after BracesFixer is not included in PSR-12 check anymore
-    ->withConfiguredRule(BracesFixer::class, ['allow_single_line_closure' => true, 'allow_single_line_anonymous_class_with_empty_body' => true])
+    ->withConfiguredRule(
+        BracesFixer::class,
+        ['allow_single_line_closure' => true, 'allow_single_line_anonymous_class_with_empty_body' => true],
+    )
     // Class, trait and interface elements must be separated with one or none blank line
     ->withConfiguredRule(ClassAttributesSeparationFixer::class, ['elements' => ['method' => 'one']])
     // Visibility MUST be declared on all properties, methods and class constants