Updated coding standard

Author: lookyman

.travis.yml

@@ -25,7 +25,10 @@ install:
   - travis_retry composer install --no-interaction --prefer-dist
 
 script:
-  - if [ "$COVERAGE" != "1" ]; then composer check; fi
+  - >
+    if [ "$COVERAGE" != "1" ]; then
+    phpenv config-rm xdebug.ini
+    && composer check; fi
   - if [ "$COVERAGE" == "1" ]; then ./vendor/bin/phpunit --coverage-clover=./coverage.xml; fi
 
 after_success:

composer.json

@@ -18,8 +18,8 @@
   "require-dev": {
     "jakub-onderka/php-parallel-lint": "^0.9.2",
     "phpstan/phpstan": "^0.6.4",
-    "lookyman/coding-standard": "^0.0.4",
-    "phpunit/phpunit": "^6.0"
+    "phpunit/phpunit": "^6.0",
+    "lookyman/coding-standard": "^0.0.5"
   },
   "autoload": {
     "psr-4": {
@@ -33,7 +33,7 @@
   },
   "scripts": {
     "lint": "parallel-lint ./src ./tests",
-    "cs": "phpcs --colors --extensions=php --encoding=utf-8 --standard=./ruleset.xml -sp ./src ./tests",
+    "cs": "phpcs --colors --extensions=php --encoding=utf-8 -sp ./src ./tests",
     "tests": "phpunit --coverage-text",
     "stan": "phpstan analyse -l 5 -c ./phpstan.neon ./src ./tests",
     "check": [

ruleset.xml phpcs.xml.distruleset.xml renamed to phpcs.xml.dist

@@ -5,12 +5,12 @@
 namespace Lookyman\PHPStan\Symfony\Rules;
 
 use Lookyman\PHPStan\Symfony\ServiceMap;
+use PHPStan\Analyser\Scope;
+use PHPStan\Rules\Rule;
 use PhpParser\Node;
 use PhpParser\Node\Arg;
 use PhpParser\Node\Expr\MethodCall;
 use PhpParser\Node\Scalar\String_;
-use PHPStan\Analyser\Scope;
-use PHPStan\Rules\Rule;
 use Symfony\Component\DependencyInjection\ContainerInterface;
 
 final class ContainerInterfacePrivateServiceRule implements Rule
@@ -49,4 +49,5 @@ public function processNode(Node $node, Scope $scope): array
 			? [\sprintf('Service "%s" is private.', $node->args[0]->value->value)]
 			: [];
 	}
+
 }

src/Rules/ContainerInterfacePrivateServiceRule.php

@@ -5,12 +5,12 @@
 namespace Lookyman\PHPStan\Symfony\Rules;
 
 use Lookyman\PHPStan\Symfony\ServiceMap;
+use PHPStan\Analyser\Scope;
+use PHPStan\Rules\Rule;
 use PhpParser\Node;
 use PhpParser\Node\Arg;
 use PhpParser\Node\Expr\MethodCall;
 use PhpParser\Node\Scalar\String_;
-use PHPStan\Analyser\Scope;
-use PHPStan\Rules\Rule;
 use Symfony\Component\DependencyInjection\ContainerInterface;
 
 final class ContainerInterfaceUnknownServiceRule implements Rule
@@ -48,4 +48,5 @@ public function processNode(Node $node, Scope $scope): array
 			? [\sprintf('Service "%s" is not registered in the container.', $node->args[0]->value->value)]
 			: [];
 	}
+
 }

src/Rules/ContainerInterfaceUnknownServiceRule.php

@@ -47,4 +47,5 @@ public function getServices(): array
 	{
 		return $this->services;
 	}
+
 }

src/ServiceMap.php

@@ -5,14 +5,14 @@
 namespace Lookyman\PHPStan\Symfony\Type;
 
 use Lookyman\PHPStan\Symfony\ServiceMap;
-use PhpParser\Node\Arg;
-use PhpParser\Node\Expr\MethodCall;
-use PhpParser\Node\Scalar\String_;
 use PHPStan\Analyser\Scope;
 use PHPStan\Reflection\MethodReflection;
 use PHPStan\Type\DynamicMethodReturnTypeExtension;
 use PHPStan\Type\ObjectType;
 use PHPStan\Type\Type;
+use PhpParser\Node\Arg;
+use PhpParser\Node\Expr\MethodCall;
+use PhpParser\Node\Scalar\String_;
 use Symfony\Component\DependencyInjection\ContainerInterface;
 
 final class ContainerInterfaceDynamicReturnTypeExtension implements DynamicMethodReturnTypeExtension
@@ -52,4 +52,5 @@ public function getTypeFromMethodCall(
 			? new ObjectType($services[$methodCall->args[0]->value->value]['class'], \false)
 			: $methodReflection->getReturnType();
 	}
+
 }