Merge pull request #46 from mcg-web/fix-ci

Fix ci
overblog · Nov 22, 2024 · 68520e0 · 68520e0
2 parents c43f73d + bb83b9d
commit 68520e0
Show file tree

Hide file tree

Showing 33 changed files with 187 additions and 83 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -1,32 +1,51 @@
 name: CI
 
 on:
-  pull_request:
-    branches:
-      - "*.*"
-      - master
+  pull_request: null
   push:
     branches:
       - "*.*"
-      - master
+      - "master"
+  schedule:
+    - cron: "0 4 * * *"
 
 jobs:
   tests:
-    runs-on: ubuntu-20.04
+    runs-on: ubuntu-22.04
     strategy:
       fail-fast: false
       matrix:
         php-version:
           - '8.0'
           - '8.1'
+          - '8.2'
+          - '8.3'
+          - '8.4'
         symfony-version:
-          - '5.3.*'
           - '5.4.*'
           - '6.0.*'
+          - '6.2.*'
+          - '6.4.*'
+          - '7.0.*'
+          - '7.1.*'
         dependencies:
           - 'lowest'
           - 'highest'
         remove-dependencies: [ '' ]
+        coverage: [ 'none' ]
+        exclude:
+          - php-version: '8.0'
+            symfony-version: '6.2.*'
+          - php-version: '8.0'
+            symfony-version: '6.4.*'
+          - php-version: '8.0'
+            symfony-version: '7.0.*'
+          - php-version: '8.0'
+            symfony-version: '7.1.*'
+          - php-version: '8.1'
+            symfony-version: '7.0.*'
+          - php-version: '8.1'
+            symfony-version: '7.1.*'
     steps:
       - name: "Checkout"
         uses: "actions/checkout@v2"
@@ -61,20 +80,21 @@ jobs:
         run: composer test
 
   coding-standard:
-    runs-on: ubuntu-20.04
+    runs-on: ubuntu-22.04
     name: Coding Standard
     steps:
       - name: "Checkout"
-        uses: "actions/checkout@v2"
+        uses: "actions/checkout@v4"
 
       - name: "Install PHP"
         uses: "shivammathur/setup-php@v2"
         with:
           tools: flex
-          php-version: "8.0"
+          php-version: "8.2"
+          coverage: "none"
 
       - name: "Install dependencies"
-        uses: ramsey/composer-install@1.3.0
+        uses: "ramsey/composer-install@v2"
 
       - name: "Check coding standard"
-        run: composer check-cs
+        run: composer run check-cs
diff --git a/.gitignore b/.gitignore
@@ -1,11 +1,13 @@
-.idea
 phpunit.xml
 /build
 /vendor
 /bin
 composer.lock
 composer.phar
-.php_cs.cache
-.php_cs
+/.php-cs-fixer.php
+/*.cache
+
+phpbench.phar
+phpbench.phar.*
+/.phpbench_storage
 php-cs-fixer.phar
-/.phpunit.result.cache
diff --git a/.php-cs-fixer.dist.php b/.php-cs-fixer.dist.php
@@ -0,0 +1,42 @@
+<?php
+
+$finder = PhpCsFixer\Finder::create()
+    ->exclude('Annotation')
+    ->name('*.php')
+    ->in([__DIR__.'/src', __DIR__.'/tests'])
+;
+
+return (new PhpCsFixer\Config())
+    ->setRules(
+        [
+            '@Symfony' => true,
+            '@PHP80Migration' => true,
+            '@PHP80Migration:risky' => true,
+            'ordered_imports' => ['imports_order' => ['class', 'function', 'const']],
+            'general_phpdoc_annotation_remove' => [
+                'annotations' => [
+                    'author',
+                    'category',
+                    'copyright',
+                    'created',
+                    'license',
+                    'package',
+                    'since',
+                    'subpackage',
+                    'version',
+                ],
+            ],
+            'fully_qualified_strict_types' => true,
+            'single_line_throw' => false,
+            'phpdoc_to_comment' => false,
+            'no_superfluous_phpdoc_tags' => ['allow_mixed' => true],
+            'global_namespace_import' => ['import_functions' => true, 'import_classes' => true, 'import_constants' => true],
+            'phpdoc_summary' => false,
+            'single_line_comment_style' => false,
+            'phpdoc_no_alias_tag' => ['replacements' => ['type' => 'var']],
+            'no_mixed_echo_print' => ['use' => 'echo'],
+        ]
+    )
+    ->setFinder($finder)
+    ->setUsingCache(true)
+    ;
diff --git a/.php_cs.dist b/.php_cs.dist
diff --git a/composer.json b/composer.json
@@ -22,24 +22,24 @@
     },
     "require": {
         "php": "^8.0",
-        "symfony/http-foundation": "^5.3.7 || ^6.0 || ^7.0",
-        "symfony/twig-bundle": "^5.3.7 || ^6.0 || ^7.0",
-        "symfony/routing": "^5.3.7 || ^6.0 || ^7.00"
+        "symfony/http-foundation": "^5.4 || ^6.0 || ^7.0",
+        "symfony/twig-bundle": "^5.4 || ^6.0 || ^7.0",
+        "symfony/routing": "^5.4 || ^6.0 || ^7.00"
     },
     "require-dev": {
         "overblog/graphql-bundle": "dev-master",
         "webonyx/graphql-php": "^15.4",
         "phpunit/phpunit": "^9.5.10",
-        "symfony/browser-kit": "^5.3 || ^6.0 || ^7.0",
-        "symfony/config": "^5.3.7 || ^6.0 || ^7.0",
-        "symfony/console": "^5.3.7 || ^6.0 || ^7.0",
-        "symfony/dependency-injection": "^5.3.7 || ^6.0 || ^7.0",
-        "symfony/dom-crawler": "^5.3.7 || ^6.0 || ^7.0",
-        "symfony/expression-language": "^5.3.7 || ^6.0 || ^7.0",
-        "symfony/framework-bundle": "^5.3.7 || ^6.0 || ^7.0",
-        "symfony/finder": "^5.3.7 || ^6.0 || ^7.0",
-        "symfony/templating": "^5.3.7 || ^6.0 || ^7.0",
-        "symfony/yaml": "^5.3.7 || ^6.0 || ^7.0",
+        "symfony/browser-kit": "^5.4 || ^6.0 || ^7.0",
+        "symfony/config": "^5.4 || ^6.0 || ^7.0",
+        "symfony/console": "^5.4 || ^6.0 || ^7.0",
+        "symfony/dependency-injection": "^5.4 || ^6.0 || ^7.0",
+        "symfony/dom-crawler": "^5.4 || ^6.0 || ^7.0",
+        "symfony/expression-language": "^5.4 || ^6.0 || ^7.0",
+        "symfony/framework-bundle": "^5.4 || ^6.0 || ^7.0",
+        "symfony/finder": "^5.4 || ^6.0 || ^7.0",
+        "symfony/templating": "^5.4 || ^6.0 || ^7.0",
+        "symfony/yaml": "^5.4 || ^6.0 || ^7.0",
         "twig/twig": "^3.3.3"
     },
     "autoload": {
@@ -50,7 +50,7 @@
     },
     "scripts": {
         "test": "bin/phpunit --color=always",
-        "install-cs": "test -f php-cs-fixer.phar || wget https://github.com/FriendsOfPHP/PHP-CS-Fixer/releases/download/v2.18.7/php-cs-fixer.phar -O php-cs-fixer.phar",
+        "install-cs": "test -f php-cs-fixer.phar || wget https://github.com/PHP-CS-Fixer/PHP-CS-Fixer/releases/download/v3.48.0/php-cs-fixer.phar -O php-cs-fixer.phar",
         "fix-cs": [
             "@install-cs",
             "@php php-cs-fixer.phar fix --diff -v --allow-risky=yes --ansi"

diff --git a/src/Config/GraphQLEndpoint/GraphQLEndpointInvalidSchemaException.php b/src/Config/GraphQLEndpoint/GraphQLEndpointInvalidSchemaException.php
@@ -1,8 +1,12 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Overblog\GraphiQLBundle\Config\GraphQLEndpoint;
 
-class GraphQLEndpointInvalidSchemaException extends \RuntimeException
+use RuntimeException;
+
+class GraphQLEndpointInvalidSchemaException extends RuntimeException
 {
     public static function forSchemaAndResolver($schemaName, $resolverClass)
     {

diff --git a/src/Config/GraphQLEndpoint/Helpers/OverblogGraphQLBundleEndpointResolver.php b/src/Config/GraphQLEndpoint/Helpers/OverblogGraphQLBundleEndpointResolver.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Overblog\GraphiQLBundle\Config\GraphQLEndpoint\Helpers;
 
 final class OverblogGraphQLBundleEndpointResolver

diff --git a/src/Config/GraphQLEndpoint/RootResolver.php b/src/Config/GraphQLEndpoint/RootResolver.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Overblog\GraphiQLBundle\Config\GraphQLEndpoint;
 
 use Overblog\GraphiQLBundle\Config\GraphiQLControllerEndpoint;

diff --git a/src/Config/GraphQLEndpoint/RouteResolver.php b/src/Config/GraphQLEndpoint/RouteResolver.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Overblog\GraphiQLBundle\Config\GraphQLEndpoint;
 
 use Overblog\GraphiQLBundle\Config\GraphiQLControllerEndpoint;

diff --git a/src/Config/GraphiQLControllerEndpoint.php b/src/Config/GraphiQLControllerEndpoint.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Overblog\GraphiQLBundle\Config;
 
 interface GraphiQLControllerEndpoint

diff --git a/src/Config/GraphiQLViewConfig.php b/src/Config/GraphiQLViewConfig.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Overblog\GraphiQLBundle\Config;
 
 final class GraphiQLViewConfig

diff --git a/src/Config/GraphiQLViewJavaScriptLibraries.php b/src/Config/GraphiQLViewJavaScriptLibraries.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Overblog\GraphiQLBundle\Config;
 
 final class GraphiQLViewJavaScriptLibraries

diff --git a/src/Controller/GraphiQLController.php b/src/Controller/GraphiQLController.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Overblog\GraphiQLBundle\Controller;
 
 use Overblog\GraphiQLBundle\Config\GraphiQLControllerEndpoint;

diff --git a/src/DependencyInjection/Compiler/Endpoints/DefaultEndpointWiringPass.php b/src/DependencyInjection/Compiler/Endpoints/DefaultEndpointWiringPass.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Overblog\GraphiQLBundle\DependencyInjection\Compiler\Endpoints;
 
 use Overblog\GraphiQLBundle\Config\GraphiQLControllerEndpoint;
@@ -9,8 +11,8 @@
 
 final class DefaultEndpointWiringPass implements CompilerPassInterface
 {
-    //@todo https://github.com/symfony/symfony/blob/master/src/Symfony/Component/DependencyInjection/Tests/Compiler/RemoveUnusedDefinitionsPassTest.php
-    public function process(ContainerBuilder $container)
+    // @todo https://github.com/symfony/symfony/blob/master/src/Symfony/Component/DependencyInjection/Tests/Compiler/RemoveUnusedDefinitionsPassTest.php
+    public function process(ContainerBuilder $container): void
     {
         $endpointDefinition = $container->getDefinition('overblog_graphiql.controller.graphql.endpoint');
 

diff --git a/src/DependencyInjection/Compiler/Endpoints/OverblogGraphQLBundleEndpointWiringPass.php b/src/DependencyInjection/Compiler/Endpoints/OverblogGraphQLBundleEndpointWiringPass.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Overblog\GraphiQLBundle\DependencyInjection\Compiler\Endpoints;
 
 use Overblog\GraphiQLBundle\Config\GraphQLEndpoint\RouteResolver;
@@ -9,7 +11,7 @@
 
 final class OverblogGraphQLBundleEndpointWiringPass implements CompilerPassInterface
 {
-    public function process(ContainerBuilder $container)
+    public function process(ContainerBuilder $container): void
     {
         $bundles = $container->getParameter('kernel.bundles_metadata');
 

diff --git a/src/DependencyInjection/Configuration.php b/src/DependencyInjection/Configuration.php
@@ -1,12 +1,16 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Overblog\GraphiQLBundle\DependencyInjection;
 
 use Overblog\GraphiQLBundle\Config\GraphQLEndpoint\Helpers\OverblogGraphQLBundleEndpointResolver;
 use Symfony\Component\Config\Definition\Builder\ArrayNodeDefinition;
 use Symfony\Component\Config\Definition\Builder\TreeBuilder;
 use Symfony\Component\Config\Definition\ConfigurationInterface;
 
+use function method_exists;
+
 final class Configuration implements ConfigurationInterface
 {
     public function getConfigTreeBuilder(): TreeBuilder
@@ -47,6 +51,6 @@ public function getConfigTreeBuilder(): TreeBuilder
     private static function getRootNodeWithoutDeprecation(TreeBuilder $builder, $name, $type = 'array')
     {
         // BC layer for symfony/config 4.1 and older
-        return \method_exists($builder, 'getRootNode') ? $builder->getRootNode() : $builder->root($name, $type);
+        return method_exists($builder, 'getRootNode') ? $builder->getRootNode() : $builder->root($name, $type);
     }
 }
diff --git a/src/DependencyInjection/OverblogGraphiQLExtension.php b/src/DependencyInjection/OverblogGraphiQLExtension.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Overblog\GraphiQLBundle\DependencyInjection;
 
 use Symfony\Component\Config\FileLocator;

diff --git a/src/OverblogGraphiQLBundle.php b/src/OverblogGraphiQLBundle.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Overblog\GraphiQLBundle;
 
 use Overblog\GraphiQLBundle\DependencyInjection\Compiler\Endpoints\DefaultEndpointWiringPass;

diff --git a/tests/Config/GraphQLEndpoint/Helpers/OverblogGraphQLBundleEndpointResolverTest.php b/tests/Config/GraphQLEndpoint/Helpers/OverblogGraphQLBundleEndpointResolverTest.php
@@ -1,13 +1,15 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Overblog\GraphiQLBundle\Tests\Config\GraphQLEndpoint\Helpers;
 
 use Overblog\GraphiQLBundle\Config\GraphQLEndpoint\Helpers\OverblogGraphQLBundleEndpointResolver;
 use PHPUnit\Framework\TestCase;
 
 final class OverblogGraphQLBundleEndpointResolverTest extends TestCase
 {
-    public function testGetDefaultRoute()
+    public function testGetDefaultRoute(): void
     {
         $defaultRoute = OverblogGraphQLBundleEndpointResolver::getByName('default');
         $this->assertSame(['overblog_graphql_endpoint'], $defaultRoute);
@@ -16,7 +18,7 @@ public function testGetDefaultRoute()
     /**
      * @dataProvider getSchemas
      */
-    public function testGetWithSchemaName($schemaName, $expectedResult)
+    public function testGetWithSchemaName($schemaName, $expectedResult): void
     {
         $route = OverblogGraphQLBundleEndpointResolver::getByName($schemaName);
         $this->assertSame($expectedResult, $route);