Added phpcs

.github/workflows/on_pull_request.yml

@@ -32,6 +32,8 @@ jobs:
         run: composer validate --strict --no-check-lock
       - name: Install Composer dependencies (with dev)
         run: composer install --no-progress --no-suggest --prefer-dist --optimize-autoloader
+      - name: PHP CodeSniffer
+        run: composer phpcs
       - name: PHPUnit
         run: composer test
       - name: Install Composer dependencies (without dev)

composer.json

@@ -18,16 +18,24 @@
             "Imponeer\\Decorators\\LogDataOutput\\": "src/"
         }
     },
+    "autoload-dev": {
+        "psr-4": {
+            "Imponeer\\Decorators\\LogDataOutput\\Tests\\": "tests/"
+        }
+    },
     "keywords": [
         "symfony",
         "console",
         "output",
         "decorator"
     ],
     "require-dev": {
-        "phpunit/phpunit": "^12"
+        "phpunit/phpunit": "^12",
+        "squizlabs/php_codesniffer": "^3.10"
     },
     "scripts": {
-        "test": "phpunit --testdox"
+        "test": "phpunit --testdox",
+        "phpcs": "phpcs --standard=PSR12 src/ tests/",
+        "phpcbf": "phpcbf --standard=PSR12 src/ tests/"
     }
 }

src/OutputDecorator.php

@@ -71,7 +71,8 @@ public function fatal(string $message, ...$params): void
      *
      * @return string
      */
-    public function renderIndentString(): string {
+    public function renderIndentString(): string
+    {
         return str_repeat(' ', $this->indent * 2);
     }
 
@@ -224,4 +225,4 @@ public function getFormatter(): OutputFormatterInterface
     {
         return $this->output->getFormatter();
     }
-}
+}

tests/OutputDecoratorTest.php

@@ -1,5 +1,7 @@
 <?php
 
+namespace Imponeer\Decorators\LogDataOutput\Tests;
+
 use Imponeer\Decorators\LogDataOutput\OutputDecorator;
 use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
@@ -18,7 +20,8 @@ protected function setUp(): void
         parent::setUp();
     }
 
-    public static function getTestData(): array {
+    public static function getTestData(): array
+    {
         return [
             'writeln (simple)' => [
                 'writeln',
@@ -87,7 +90,8 @@ public static function getTestData(): array {
         ];
     }
 
-    private function useDecoratorMethod(string $method, string $text, array $params): string {
+    private function useDecoratorMethod(string $method, string $text, array $params): string
+    {
         if (empty($params)) {
             $this->decorator->$method($text);
         } else {
@@ -99,7 +103,8 @@ private function useDecoratorMethod(string $method, string $text, array $params)
     }
 
     #[DataProvider('getTestData')]
-    public function testIncrIndent(string $method, string $text, array $args = [], ?string $shouldReturn = null): void {
+    public function testIncrIndent(string $method, string $text, array $args = [], ?string $shouldReturn = null): void
+    {
         if ($shouldReturn === null) {
             $shouldReturn = $text;
         }
@@ -136,7 +141,8 @@ public function testDecrIndent(string $method, string $text, array $args = [], ?
     }
 
     #[DataProvider('getTestData')]
-    public function testResetIncr(string $method, string $text, array $args = [], ?string $shouldReturn = null): void {
+    public function testResetIncr(string $method, string $text, array $args = [], ?string $shouldReturn = null): void
+    {
         if ($shouldReturn === null) {
             $shouldReturn = $text;
         }
@@ -152,5 +158,4 @@ public function testResetIncr(string $method, string $text, array $args = [], ?s
         $buffer = $this->useDecoratorMethod($method, $text, $args);
         $this->assertSame($shouldReturn . PHP_EOL, $buffer);
     }
-
-}
+}