From c4afc7a923c56f3d80b10039d1ecc40efdf46d92 Mon Sep 17 00:00:00 2001
From: Alejandro Celaya <alejandro@alejandrocelaya.com>
Date: Mon, 26 Nov 2018 20:45:44 +0100
Subject: [PATCH] Updated project to use external coding standard

---
 CHANGELOG.md  |  1 +
 composer.json |  3 +--
 phpcs.xml     | 48 +++++-------------------------------------------
 3 files changed, 7 insertions(+), 45 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 8047d86d..e4d50f30 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -17,6 +17,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 #### Changed
 
 * [#267](https://github.com/shlinkio/shlink/issues/267) API responses and the CLI interface is no longer translated and uses english always. Only not found error templates are still translated.
+* [#289](https://github.com/shlinkio/shlink/issues/289) Extracted coding standard rules to a separated package.
 
 #### Deprecated
 
diff --git a/composer.json b/composer.json
index 8c79a730..898ce0ed 100644
--- a/composer.json
+++ b/composer.json
@@ -56,8 +56,7 @@
         "phpstan/phpstan": "^0.10.0",
         "phpunit/phpcov": "^5.0",
         "phpunit/phpunit": "^7.3",
-        "slevomat/coding-standard": "^4.0",
-        "squizlabs/php_codesniffer": "^3.2.3",
+        "shlinkio/php-coding-standard": "~1.0.0",
         "symfony/dotenv": "^4.0",
         "symfony/var-dumper": "^4.0",
         "zendframework/zend-component-installer": "^2.1",
diff --git a/phpcs.xml b/phpcs.xml
index 40a10e91..ee4221b4 100644
--- a/phpcs.xml
+++ b/phpcs.xml
@@ -1,53 +1,13 @@
 <?xml version="1.0"?>
 <ruleset name="Coding standard">
     <description>Coding standard</description>
-    <config name="installed_paths" value="../../slevomat/coding-standard"/>
 
     <!-- display progress -->
-    <arg value="p"/>
-    <arg name="colors"/>
+    <arg value="p" />
+    <arg name="colors" />
 
     <!-- inherit rules from: -->
-    <rule ref="PSR2"/>
-    <rule ref="Generic.Arrays.DisallowLongArraySyntax"/>
-    <!-- Make sure string concatenations use 1 space between every element -->
-    <rule ref="Squiz.Strings.ConcatenationSpacing">
-        <properties>
-            <property name="spacing" value="1"/>
-            <property name="ignoreNewlines" value="true"/>
-        </properties>
-    </rule>
-    <rule ref="Squiz.WhiteSpace.SuperfluousWhitespace">
-        <properties>
-            <property name="ignoreBlankLines" value="false"/>
-        </properties>
-    </rule>
-    <rule ref="Squiz.Strings.DoubleQuoteUsage"/>
-    <rule ref="SlevomatCodingStandard.TypeHints.DeclareStrictTypes">
-        <properties>
-            <property name="newlinesCountBetweenOpenTagAndDeclare" value="1"/>
-            <property name="spacesCountAroundEqualsSign" value="0"/>
-        </properties>
-    </rule>
-    <rule ref="SlevomatCodingStandard.Namespaces.UnusedUses">
-        <properties>
-            <property name="searchAnnotations" value="true"/>
-        </properties>
-    </rule>
-    <!-- Force strict comparison with === or !== instead of == or != -->
-    <rule ref="SlevomatCodingStandard.ControlStructures.DisallowEqualOperators"/>
-    <rule ref="SlevomatCodingStandard.Arrays.TrailingArrayComma"/>
-    <rule ref="SlevomatCodingStandard.Namespaces.AlphabeticallySortedUses"/>
-    <rule ref="SlevomatCodingStandard.TypeHints.ReturnTypeHintSpacing"/>
-    <!-- Enforce all global namespace classes, functions and constants to be explicitly imported -->
-    <rule ref="SlevomatCodingStandard.Namespaces.ReferenceUsedNamesOnly">
-        <properties>
-            <property name="allowFallbackGlobalFunctions" value="false"/>
-            <property name="allowFallbackGlobalConstants" value="false"/>
-        </properties>
-    </rule>
-    <!-- Require comments with single line written as one-liners -->
-    <rule ref="SlevomatCodingStandard.Commenting.RequireOneLinePropertyDocComment"/>
+    <rule ref="Shlinkio" />
 
     <!-- Paths to check -->
     <file>bin</file>
@@ -55,6 +15,8 @@
     <file>data/migrations</file>
     <file>config</file>
     <file>public/index.php</file>
+
+    <!-- Paths to exclude -->
     <exclude-pattern>config/params/*</exclude-pattern>
     <exclude-pattern>public/index.php</exclude-pattern>
 </ruleset>