CI: improvement Travis, added phpstan, updated tester

.travis.yml

@@ -1,51 +1,59 @@
 language: php
-
 php:
-  - 5.6
-  - 7.0
-  - 7.1
-  - hhvm
-
-matrix:
-  fast_finish: true
+    - 5.6
+    - 7.0
+    - 7.1
+    - 7.2
 
-  allow_failures:
-      - php: hhvm
+before_install:
+  # turn off XDebug
+  - phpenv config-rm xdebug.ini || return 0
 
-  include:
-    - php: 5.6
-      env: COMPOSER_FLAG=--prefer-lowest
-    - php: 5.6
-      env: COMPOSER_FLAG=--prefer-stable
-    - php: 7.0
-      env: COMPOSER_FLAG=--prefer-lowest
-    - php: 7.0
-      env: COMPOSER_FLAG=--prefer-stable
-    - php: 7.1
-      env: COMPOSER_FLAG=--prefer-lowest
-    - php: 7.1
-      env: COMPOSER_FLAG=--prefer-stable
-
-before_script:
+install:
   # Composer
-  - travis_retry composer install --no-interaction
-  # Coverage
-  - if [[ "$TRAVIS_PHP_VERSION" == "7.1" && "$COMPOSER_FLAG" == "" ]]; then COVERAGE=1; fi
+  - travis_retry composer install --no-progress --prefer-dist
 
 script:
-  # Quality Assurance
-  - travis_retry composer qa
-
-  # Nette\Tester
+  # Nette/Tester
   - composer run-script tester
 
-  # Nette\Tester + CodeCoverage
-  - if [ "$COVERAGE" != "" ]; then composer tester-coverage; fi
+jobs:
+  include:
+    - env: title="Lowest Dependencies 5.6"
+      php: 5.6
+      install:
+        - travis_retry composer update --no-progress --prefer-dist --prefer-lowest
+      script:
+        - composer run-script tester
 
-after_script:
-  # Coverage (Coveralls)
-  - if [ "$COVERAGE" != "" ]; then wget https://github.com/satooshi/php-coveralls/releases/download/v1.0.1/coveralls.phar; fi
-  - if [ "$COVERAGE" != "" ]; then php coveralls.phar --verbose --config tests/.coveralls.yml; fi
+    - env: title="Lowest Dependencies 7.1"
+      php: 7.1
+      install:
+        - travis_retry composer update --no-progress --prefer-dist --prefer-lowest
+      script:
+        - composer run-script tester
+
+    - stage: Quality Assurance
+      php: 7.1
+      script:
+        - composer run-script qa
+
+    - stage: Test Coverage
+      php: 7.1
+      script:
+        - composer run-script coverage
+      after_script:
+        - wget https://github.com/satooshi/php-coveralls/releases/download/v1.0.1/coveralls.phar
+        - php coveralls.phar --verbose --config tests/.coveralls.yml
+
+    - stage: Phpstan
+      php: 7.1
+      script:
+        - composer run-script phpstan-install
+        - composer run-script phpstan
+
+  allow_failures:
+    - stage: Test Coverage
 
 after_failure:
   # Print *.actual content

composer.json

@@ -19,7 +19,7 @@
   },
   "require-dev": {
     "ninjify/qa": "~0.4.0",
-    "ninjify/nunjuck": "~0.1.4"
+    "ninjify/nunjuck": "~0.2.0"
   },
   "autoload": {
     "psr-4": {
@@ -32,10 +32,18 @@
       "codesniffer src tests"
     ],
     "tester": [
-      "tester -s -p php --colors 1 -c tests/php-unix.ini tests/cases"
+      "tester -s -p php --colors 1 -C tests/cases"
     ],
-    "tester-coverage": [
-      "tester -s -p php --colors 1 -c tests/php-unix.ini -d extension=xdebug.so --coverage ./coverage.xml --coverage-src ./src tests/cases"
+    "coverage": [
+      "tester -s -p php --colors 1 -C -d extension=xdebug.so --coverage ./coverage.xml --coverage-src ./src tests/cases"
+    ],
+    "phpstan-install": [
+      "mkdir -p temp/phpstan",
+      "composer require -d temp/phpstan phpstan/phpstan:0.8.5",
+      "composer require -d temp/phpstan phpstan/phpstan-nette:0.8.3"
+    ],
+    "phpstan": [
+      "temp/phpstan/vendor/bin/phpstan analyse -l 7 -c phpstan.neon src"
     ]
   }
 }

phpstan.neon

@@ -0,0 +1,5 @@
+includes:
+	- temp/phpstan/vendor/phpstan/phpstan-nette/extension.neon
+	- temp/phpstan/vendor/phpstan/phpstan-nette/rules.neon
+
+parameters:

src/DI/MigrationsExtension.php

@@ -19,7 +19,7 @@
 final class MigrationsExtension extends CompilerExtension
 {
 
-	/** @var string[] */
+	/** @var mixed[] */
 	private $defaults = [
 		'table' => 'doctrine_migrations',
 		'column' => 'version',

src/Helper/ConfigurationHelper.php

@@ -11,7 +11,7 @@
 class ConfigurationHelper extends BaseConfigurationHelper
 {
 
-	/** @var Configuration */
+	/** @var Configuration|NULL */
 	private $configuration;
 
 	/**
@@ -31,8 +31,9 @@ public function __construct(Connection $connection = NULL, Configuration $config
 	 */
 	public function getMigrationConfig(InputInterface $input, OutputWriter $outputWriter)
 	{
-		$this->configuration->setOutputWriter($outputWriter);
-
+		if ($this->configuration !== NULL) {
+			$this->configuration->setOutputWriter($outputWriter);
+		}
 		return $this->configuration;
 	}