cleanup CS (#183)

Author: LordSimal

.gitignore

@@ -5,7 +5,6 @@
 /tags
 /composer.lock
 /phpunit.xml
-/phpcs.xml
 /tools
 /vendor
 *.mo
@@ -16,6 +15,7 @@ error.log
 /tests/test_app/config/TestsQueue/schema-dump-test.lock
 /tests/test_app/Plugin/TestBlog/config/Queue/*
 .phpunit.cache
+.phpcs.cache
 
 # IDE and editor specific files #
 #################################

composer.json

@@ -60,8 +60,8 @@
             "@cs-check",
             "@test"
         ],
-        "cs-check": "phpcs --colors -p src/ tests/",
-        "cs-fix": "phpcbf --colors -p src/ tests/",
+        "cs-check": "phpcs",
+        "cs-fix": "phpcbf",
         "stan": "@phpstan",
         "phpstan": "tools/phpstan analyse",
         "stan-baseline": "tools/phpstan --generate-baseline",

phpcs.xml

@@ -0,0 +1,12 @@
+<?xml version="1.0"?>
+<ruleset name="CakePHP PluginInstaller">
+    <file>src/</file>
+    <file>tests/</file>
+    <rule ref="CakePHP" />
+    <arg value="nps"/>
+    <arg name="colors"/>
+    <arg name="parallel" value="4"/>
+    <arg name="cache" value=".phpcs.cache"/>
+
+    <exclude-pattern>tests/comparisons/*</exclude-pattern>
+</ruleset>

rector.php

@@ -25,6 +25,7 @@
         DocblockReturnArrayFromDirectArrayInstanceRector::class => [
             __DIR__ . '/src/Mailer/Transport/QueueTransport.php',
         ],
+        'tests/TestCase/Queue/ProcessorTest.php',
     ])
     ->withParallel()
     ->withPreparedSets(

src/Job/SendMailJob.php

@@ -38,7 +38,6 @@ public function execute(Message $message): ?string
         try {
             $transportClassName = $message->getArgument('transport');
             $config = $message->getArgument('config', []);
-            /** @var \Cake\Mailer\AbstractTransport $transport */
             $transport = $this->getTransport($transportClassName, $config);
 
             $emailMessage = new MailerMessage();