Improve php-cs-fixer config

Author: crodas

.github/workflows/main.yml

@@ -8,7 +8,7 @@ jobs:
       fail-fast: false
       matrix:
         operating-system: [ubuntu-latest, windows-latest, macos-latest]
-        php-versions: ['7.2', '7.3', '7.4', '8.0']
+        php-versions: ['7.3', '7.4', '8.0']
     steps:
       - name: Checkout
         uses: actions/checkout@v2

.php_cs.dist

@@ -36,14 +36,14 @@ $config
         '@PHPUnit75Migration:risky' => true,
         '@Symfony' => true,
         '@PhpCsFixer' => true,
-        '@PhpCsFixer:risky' => true,
         'binary_operator_spaces' => ['default' => 'align_single_space'],
         'align_multiline_comment' => true,
         'array_syntax' => ['syntax' => 'short'],
         'array_indentation' => true,
         'blank_line_before_return' => true,
         'header_comment' => ['header' => $header],
         'list_syntax' => ['syntax' => 'long'],
+        'concat_space' => ['spacing' => 'one']
     ])
     ->setFinder($finder)
 ;

Makefile

@@ -6,6 +6,6 @@ setup:
 build:
 	./vendor/bin/phplemon src/SQLParser/Parser.y || exit 0
 	php build.php
-	./vendor/bin/php-cs-fixer fix src
+	./vendor/bin/php-cs-fixer fix
 test:
 	phpunit --coverage-html coverage

build.php

@@ -1,5 +1,29 @@
 <?php
 
+/*
+ * The MIT License (MIT)
+ *
+ * Copyright (c) 2015-2021 César Rodas
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a copy
+ * of this software and associated documentation files (the "Software"), to deal
+ * in the Software without restriction, including without limitation the rights
+ * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+ * copies of the Software, and to permit persons to whom the Software is
+ * furnished to do so, subject to the following conditions:
+ * -
+ * The above copyright notice and this permission notice shall be included in
+ * all copies or substantial portions of the Software.
+ * -
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+ * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
+ * THE SOFTWARE.
+ */
+
 require __DIR__ . '/vendor/autoload.php';
 
 use SQLParser\Lexer;
@@ -18,9 +42,8 @@
 
 class ReservedWords
 {
-    ' . $constants ."
+    ' . $constants . "
     public static \$words = [\n" . $array . '];
 }';
 
-
 file_put_contents(__DIR__ . '/src/SQL/ReservedWords.php', $code);

composer.json

@@ -17,7 +17,7 @@
         ]
     },
     "requires": {
-        "php": "^7.0 || ^8.0"
+        "php": ">=7.3 || ^8.0"
     },
     "require-dev": {
         "crodas/phpunit-compat": "^1.7",

src/SQL/Statement.php

@@ -178,7 +178,7 @@ public function setOptions(array $mods)
             }
 
             if (\count($walk) > 1) {
-                throw new RuntimeException('Invalid usage of '.implode(', ', $walk));
+                throw new RuntimeException('Invalid usage of ' . implode(', ', $walk));
             }
         }
 

src/SQL/TableDiff.php

@@ -128,7 +128,7 @@ public function getTable($sql)
     {
         $stmts = $this->parser->parse($sql);
         if (!($stmts[0] instanceof Table)) {
-            throw new InvalidArgumentException('Expecting a CREATE TABLE Statement, got '.\get_class($stmts[0]).' class');
+            throw new InvalidArgumentException('Expecting a CREATE TABLE Statement, got ' . \get_class($stmts[0]) . ' class');
         }
 
         $table = array_shift($stmts);