Codestyle with phpcs (#29)

* Add phpcs

Co-authored-by: Sara Gfrörer <[email protected]>

Author: srmnq

composer.json

@@ -20,11 +20,14 @@
     "check": [
       "@analyze",
       "@tests",
-      "@tests-spec"
+      "@tests-spec",
+      "@cs-check"
     ],
     "tests": "phpunit tests",
     "analyze": "phpstan analyse --level max",
-    "tests-spec": "phpspec run --no-interaction"
+    "tests-spec": "phpspec run --no-interaction",
+    "cs-check": "phpcs --parallel=50",
+    "cs-fix": "phpcbf"
   },
   "require-dev": {
     "phpspec/phpspec": "^6.2",

phpcs.xml.dist

@@ -0,0 +1,21 @@
+<?xml version = "1.0"?>
+<ruleset name="PSR12 coding standard">
+    <!-- display progress -->
+    <arg value="p"/>
+    <arg name="colors"/>
+
+    <!-- inherit rules from: -->
+    <rule ref="PSR12">
+        <exclude name="Generic.Files.LineLength"/>
+    </rule>
+    <rule ref="Generic.Arrays.DisallowLongArraySyntax"/>
+    <rule ref="Squiz.WhiteSpace.SuperfluousWhitespace">
+        <properties>
+            <property name="ignoreBlankLines" value="false"/>
+        </properties>
+    </rule>
+
+    <!-- Paths to check -->
+    <file>src</file>
+    <file>tests</file>
+</ruleset>

src/Check/NumericRangeCheck.php

@@ -48,7 +48,7 @@ public function check(string $key, $value): ResultInterface
         $result = $this->child->check($key, $value);
 
         if (!$result->isValid()) {
-           return $result;
+            return $result;
         }
 
         if ($this->lowerBound > $value) {

src/Type/NullableType.php

@@ -18,7 +18,7 @@ public function __construct(TypeInterface $child)
 
     public function check(string $key, $value): ResultInterface
     {
-        if($value === null) {
+        if ($value === null) {
             return Result::valid();
         }