Merge pull request #417 from craue/gha

migrate from Travis CI to GitHub Actions

.gitattributes

@@ -1,8 +1,7 @@
 /Tests export-ignore
 /.gitattributes export-ignore
+/.github export-ignore
 /.gitignore export-ignore
-/.travis_install_dependencies.sh export-ignore
-/.travis.yml export-ignore
 /phpstan-config.neon export-ignore
 /phpstan.neon.dist export-ignore
 /phpunit.xml.dist export-ignore

.github/workflows/composer-lint.yml

@@ -0,0 +1,31 @@
+name: Composer Lint
+
+on:
+  pull_request: ~
+  push: ~
+
+jobs:
+  composer-lint:
+    name: Composer Lint
+    runs-on: ubuntu-latest
+
+    strategy:
+      matrix:
+        php:
+          - '8.2'
+
+    steps:
+      - name: checkout
+        uses: actions/checkout@v3
+
+      - name: setup PHP
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: ${{ matrix.php }}
+          coverage: none
+          tools: composer:v2, composer-normalize:2
+        env:
+          COMPOSER_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+
+      - name: run composer-normalize
+        run: composer-normalize --dry-run

.github/workflows/static-analysis.yml

@@ -0,0 +1,31 @@
+name: Static Analysis
+
+on:
+  pull_request: ~
+  push: ~
+
+jobs:
+  static-analysis-phpstan:
+    name: Static Analysis with PHPStan
+    runs-on: ubuntu-latest
+
+    strategy:
+      matrix:
+        php:
+          - '8.2'
+
+    steps:
+      - name: checkout
+        uses: actions/checkout@v3
+
+      - name: setup PHP
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: ${{ matrix.php }}
+          coverage: none
+
+      - name: install dependencies with Composer
+        uses: ramsey/composer-install@v2
+
+      - name: run PHPStan
+        run: vendor/bin/phpstan analyse

.github/workflows/tests.yml

@@ -0,0 +1,106 @@
+name: Tests
+
+on:
+  pull_request: ~
+  push: ~
+
+concurrency:
+  group: ${{ github.workflow }} @ ${{ github.head_ref || github.ref }}
+  cancel-in-progress: true
+
+jobs:
+  tests:
+    name: PHP ${{ matrix.php }} - ${{ matrix.dependencies || format('Symfony {0}', matrix.symfony) }}${{ (matrix.note && format(' ({0})', matrix.note)) || '' }}${{ (matrix.allow-failure && ' 🔕') || '' }}
+    runs-on: ubuntu-latest
+    continue-on-error: ${{ matrix.allow-failure || false }}
+    env:
+      SYMFONY_REQUIRE: ${{ matrix.symfony }}
+      SYMFONY_DEPRECATIONS_HELPER: ${{ matrix.symfony-deprecations }}
+      PARAM_DB_DRIVER: ${{ matrix.db-driver }}
+
+    strategy:
+      fail-fast: false
+      matrix:
+        include:
+          -
+            php: '7.3'
+            dependencies: lowest
+          -
+            php: '8.2'
+            dependencies: highest
+            symfony-deprecations: max[indirect]=5722
+          -
+            php: '7.3'
+            symfony: '4.4.*'
+          -
+            php: '7.3'
+            symfony: '4.4.*'
+            note: sqlite
+            db-driver: pdo_sqlite
+          -
+            php: '7.4'
+            symfony: '5.4.*'
+          # TODO remove as soon as Symfony >= 5.0 is required
+          -
+            php: '7.4'
+            symfony: '5.4.*'
+            note: contao-polyfill-symfony
+            contao-polyfill-symfony: true
+          -
+            php: '8.0'
+            symfony: '5.4.*'
+          -
+            php: '8.1'
+            symfony: '5.4.*'
+          -
+            php: '8.1'
+            symfony: '6.3.*'
+            symfony-deprecations: max[indirect]=5722
+          -
+            php: '8.2'
+            symfony: '6.4.*'
+            stability: dev
+            allow-failure: true
+
+    steps:
+      - name: checkout
+        uses: actions/checkout@v3
+
+      - name: setup PHP
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: ${{ matrix.php }}
+          extensions: mysql, redis
+          coverage: pcov
+          ini-values: memory_limit=-1
+          tools: flex
+        env:
+          fail-fast: true # interrupt on extension setup error
+
+      # TODO remove as soon as Symfony >= 5.0 is required
+      - name: require contao/polyfill-symfony
+        if: matrix.contao-polyfill-symfony == true
+        run: composer require --no-update --dev contao/polyfill-symfony
+
+      - name: use dev dependencies
+        if: matrix.stability == 'dev'
+        run: composer config minimum-stability dev
+
+      - name: install dependencies with Composer
+        uses: ramsey/composer-install@v2
+        with:
+          dependency-versions: ${{ matrix.dependencies }}
+          composer-options: --prefer-dist
+
+      - name: run PHPUnit
+        run: vendor/bin/phpunit -v --coverage-clover build/logs/clover.xml
+
+      - name: upload code coverage data
+        if: github.repository == 'craue/CraueFormFlowBundle'
+        env:
+          COVERALLS_REPO_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+          COVERALLS_FLAG_NAME: PHP ${{ matrix.php }} - ${{ matrix.dependencies || format('Symfony {0}', matrix.symfony) }}${{ (matrix.note && format(' ({0})', matrix.note)) || '' }}
+        run: |
+          unset SYMFONY_REQUIRE
+          composer global require php-coveralls/php-coveralls
+          php-coveralls --coverage_clover=build/logs/clover.xml -v

CraueFormFlowBundle.php

@@ -16,7 +16,7 @@
 class CraueFormFlowBundle extends Bundle {
 
 	/**
-	 * {@inheritDoc}
+	 * @return void
 	 */
 	public function boot() {
 		/*

DependencyInjection/Compiler/LegacySessionCompilerPass.php

@@ -18,7 +18,7 @@
 class LegacySessionCompilerPass implements CompilerPassInterface {
 
 	/**
-	 * {@inheritDoc}
+	 * @return void
 	 */
 	public function process(ContainerBuilder $container) : void {
 		// TODO remove as soon as Symfony >= 5.3 is required

DependencyInjection/CraueFormFlowExtension.php

@@ -21,7 +21,7 @@ class CraueFormFlowExtension extends Extension implements CompilerPassInterface
 	const FORM_FLOW_TAG = 'craue.form.flow';
 
 	/**
-	 * {@inheritDoc}
+	 * @return void
 	 */
 	public function load(array $config, ContainerBuilder $container) {
 		$loader = new XmlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));
@@ -33,7 +33,7 @@ public function load(array $config, ContainerBuilder $container) {
 	}
 
 	/**
-	 * {@inheritDoc}
+	 * @return void
 	 */
 	public function process(ContainerBuilder $container) {
 		$baseFlowDefinitionMethodCalls = $container->getDefinition('craue.form.flow')->getMethodCalls();

Form/Extension/FormFlowFormExtension.php

@@ -17,7 +17,7 @@
 class FormFlowFormExtension extends AbstractTypeExtension {
 
 	/**
-	 * {@inheritDoc}
+	 * @return string
 	 */
 	public function getExtendedType() {
 		return FormType::class;
@@ -28,7 +28,7 @@ public static function getExtendedTypes() : iterable {
 	}
 
 	/**
-	 * {@inheritDoc}
+	 * @return void
 	 */
 	public function configureOptions(OptionsResolver $resolver) {
 		$resolver->setDefined([
@@ -40,7 +40,7 @@ public function configureOptions(OptionsResolver $resolver) {
 	}
 
 	/**
-	 * {@inheritDoc}
+	 * @return void
 	 */
 	public function buildForm(FormBuilderInterface $builder, array $options) {
 		if (array_key_exists('flow_instance', $options) && array_key_exists('flow_instance_key', $options)) {

Form/Extension/FormFlowHiddenFieldExtension.php

@@ -17,7 +17,7 @@
 class FormFlowHiddenFieldExtension extends AbstractTypeExtension {
 
 	/**
-	 * {@inheritDoc}
+	 * @return string
 	 */
 	public function getExtendedType() {
 		return HiddenType::class;
@@ -28,7 +28,7 @@ public static function getExtendedTypes() : iterable {
 	}
 
 	/**
-	 * {@inheritDoc}
+	 * @return void
 	 */
 	public function configureOptions(OptionsResolver $resolver) {
 		$resolver->setDefined([
@@ -38,7 +38,7 @@ public function configureOptions(OptionsResolver $resolver) {
 	}
 
 	/**
-	 * {@inheritDoc}
+	 * @return void
 	 */
 	public function finishView(FormView $view, FormInterface $form, array $options) {
 		if (array_key_exists('flow_instance_key', $options) && $view->vars['name'] === $options['flow_instance_key']) {