Introduce coding standard checking

.gitignore

@@ -1,3 +1,4 @@
 /vendor/
-.phpunit.result.cache
-composer.lock
+/.phpcs-cache
+/.phpunit.result.cache
+/composer.lock

Makefile

@@ -1,6 +1,7 @@
 check: vendor
 	vendor/bin/phpstan analyse
 	vendor/bin/phpunit --testdox
+	vendor/bin/phpcs
 	composer validate
 
 vendor: composer.lock

composer.json

@@ -8,8 +8,10 @@
         "behat/behat": "^3.0"
     },
     "require-dev": {
+        "nicwortel/coding-standard": "^2.0",
         "phpstan/phpstan": "^0.12.10",
         "phpunit/phpunit": "^9.0",
+        "squizlabs/php_codesniffer": "^3.5",
         "symfony/process": "^5.0"
     },
     "config": {

phpcs.xml

@@ -0,0 +1,21 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<ruleset xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:noNamespaceSchemaLocation="vendor/squizlabs/php_codesniffer/phpcs.xsd">
+    <arg name="basepath" value="."/>
+    <arg name="cache" value=".phpcs-cache"/>
+    <arg name="colors"/>
+    <arg name="extensions" value="php"/>
+
+    <!-- Show progress and sniff names -->
+    <arg value="p" />
+
+    <file>src/</file>
+
+    <rule ref="NicWortel"/>
+
+    <rule ref="SlevomatCodingStandard.TypeHints.PropertyTypeHint">
+        <properties>
+            <property name="enableNativeTypeHint" value="false"/>
+        </properties>
+    </rule>
+</ruleset>

src/Extension.php

@@ -1,4 +1,5 @@
 <?php
+
 declare(strict_types=1);
 
 namespace NicWortel\BehatUnusedStepDefinitionsExtension;

src/UnusedStepDefinitionsChecker.php

@@ -1,4 +1,5 @@
 <?php
+
 declare(strict_types=1);
 
 namespace NicWortel\BehatUnusedStepDefinitionsExtension;
@@ -9,7 +10,10 @@
 use Behat\Behat\EventDispatcher\Event\AfterStepTested;
 use Behat\Testwork\EventDispatcher\Event\AfterSuiteTested;
 use Symfony\Component\EventDispatcher\EventSubscriberInterface;
+
 use function array_diff;
+use function sprintf;
+
 use const PHP_EOL;
 
 final class UnusedStepDefinitionsChecker implements EventSubscriberInterface
@@ -72,10 +76,10 @@ public function checkUnusedStepDefinitions(AfterSuiteTested $event): void
 
         foreach ($unusedDefinitions as $unusedDefinition) {
             echo sprintf(
-                    'Unused definition: %s %s',
-                    $unusedDefinition->getType(),
-                    $unusedDefinition->getPattern()
-                ) . PHP_EOL;
+                'Unused definition: %s %s',
+                $unusedDefinition->getType(),
+                $unusedDefinition->getPattern()
+            ) . PHP_EOL;
         }
     }
 }