Merge pull request BitBagCommerce#82 from stefandoorn/phpstan

Add PHPStan at maximum level

.travis.yml

@@ -65,6 +65,7 @@ before_script:
 
 script:
     - composer validate --strict
+    - bin/phpstan.phar analyse -c phpstan.neon -l max src/
 
     - bin/behat --strict -vvv --no-interaction || bin/behat --strict -vvv --no-interaction --rerun
 

composer.json

@@ -5,12 +5,10 @@
     "license": "MIT",
     "require": {
         "php": "^7.1",
-
+        
         "sylius/sylius": "^1.0.4"
     },
     "require-dev": {
-        "phpspec/phpspec": "^3.2",
-        "phpunit/phpunit": "^5.6",
         "behat/behat": "^3.3",
         "behat/mink": "^1.7",
         "behat/mink-browserkit-driver": "^1.3",
@@ -22,9 +20,12 @@
         "friends-of-behat/symfony-extension": "^1.0",
         "friends-of-behat/variadic-extension": "^1.0",
         "lakion/mink-debug-extension": "^1.2.3",
+        "phpspec/phpspec": "^3.2",
+        "phpstan/phpstan-shim": "^0.9.2",
+        "phpunit/phpunit": "^5.6",
         "se/selenium-server-standalone": "^2.52",
-        "symplify/easy-coding-standard": "^2.4",
-        "sylius-labs/coding-standard": "^1.0"
+        "sylius-labs/coding-standard": "^1.0",
+        "symplify/easy-coding-standard": "^2.4"
     },
     "prefer-stable": true,
     "autoload": {

phpstan.neon

@@ -0,0 +1,11 @@
+parameters:
+    excludes_analyse:
+        # Makes PHPStan crash
+        - 'src/DependencyInjection/Configuration.php'
+
+        # Test dependencies
+        - 'tests/Application/app/**.php'
+        - 'tests/Application/src/**.php'
+
+    ignoreErrors:
+        - '/Parameter #1 $configuration of method Symfony\Component\DependencyInjection\Extension\Extension::processConfiguration() expects Symfony\Component\Config\Definition\ConfigurationInterface, Symfony\Component\Config\Definition\ConfigurationInterface|null given./'