Merge pull request #58 from simPod/phpstan

Add PHPStan

Author: isublimity

.travis.yml

@@ -54,6 +54,10 @@ jobs:
         - composer config minimum-stability dev
         - travis_retry composer update -n --prefer-dist
 
+    - stage: Code Quality
+      install: travis_retry composer install --prefer-dist
+      script: vendor/bin/phpstan analyse -c phpstan.neon src
+
 after_script:
   # upload clover.xml file to Scrutinizer to analyze it
   - |

composer.json

@@ -17,7 +17,8 @@
     "ext-curl": "*"
   },
   "require-dev": {
-    "phpunit/phpunit": "^7.0"
+    "phpunit/phpunit": "^7.0",
+    "phpstan/phpstan": "^0.9.2"
   },
   "autoload": {
     "psr-4": {

phpstan.neon

@@ -0,0 +1,3 @@
+includes:
+    # Enable Level 1
+    - vendor/phpstan/phpstan/conf/config.level1.neon

src/Exception/Exception.php

@@ -35,6 +35,9 @@ class Statement
      */
     private $query;
 
+    /** @var mixed */
+    private $format;
+
     /**
      * @var string
      */

src/Statement.php

@@ -3,7 +3,6 @@
 namespace ClickHouseDB\Tests;
 
 use ClickHouseDB\Client;
-use ClickHouseDB\Exception;
 use ClickHouseDB\Exception\DatabaseException;
 use ClickHouseDB\Exception\QueryException;
 use ClickHouseDB\Query\WhereInFile;
@@ -24,9 +23,6 @@ class ClientTest extends TestCase
 {
     use WithClient;
 
-    /**
-     * @throws Exception
-     */
     public function setUp()
     {
         date_default_timezone_set('Europe/Moscow');