[v5] PHP 8.1, Symfony 6.4, phpstan

.github/workflows/continuous-integration.yml

@@ -19,42 +19,29 @@ env:
 jobs:
   phpunit:
     name: "PHPUnit"
-    runs-on: "ubuntu-22.04"
+    runs-on: "ubuntu-latest"
 
     strategy:
       fail-fast: false
       matrix:
-        include:
-          - php-version: 7.2
-            composer-flags: "--prefer-lowest"
+        php:
+          - "8.1"
+          - "8.2"
+          - "8.3"
+        symfony-require:
+          - "^6.4"
+          - "^7.0"
+        dependencies:
+          - "lowest"
+          - "highest"
+        doctrine-annotations:
+          - true
+          - false
+        exclude:
+          - symfony-require: "^7.0"
             doctrine-annotations: true
-          - php-version: 7.3
-            symfony-require: "5.4.*"
-            doctrine-annotations: true
-          - php-version: 7.4
-            symfony-require: "5.4.*"
-            doctrine-annotations: true
-          - php-version: 8.0
-            symfony-require: "5.4.*"
-            doctrine-annotations: true
-          - php-version: 8.1
-            symfony-require: "5.4.*"
-            doctrine-annotations: true
-          - php-version: 8.3
-            symfony-require: "5.4.*"
-            doctrine-annotations: true
-          - php-version: 8.1
-            symfony-require: "6.3.*"
-            doctrine-annotations: true
-          - php-version: 8.3
-            symfony-require: "6.3.*"
-            doctrine-annotations: true
-          - php-version: 8.2
-            symfony-require: "7.0.*"
-            doctrine-annotations: false
-          - php-version: 8.3
-            symfony-require: "7.0.*"
-            doctrine-annotations: false
+          - php: "8.1"
+            symfony-require: "^7.0"
 
     steps:
       - name: "Checkout"
@@ -86,14 +73,14 @@ jobs:
           composer require doctrine/annotations --no-update
 
       - name: Remove packages not compatible symfony 7
-        if: matrix.symfony-require == '7.0.*'
+        if: matrix.symfony-require == '^7.0'
         run: |
           composer remove friendsofsymfony/rest-bundle sensio/framework-extra-bundle --no-update --dev
 
       - name: "Install dependencies with composer"
+        uses: "ramsey/composer-install@v2"
         env:
           SYMFONY_REQUIRE: "${{ matrix.symfony-require }}"
-        run: composer update --no-interaction --no-progress ${{ matrix.composer-flags }}
 
       - name: "PHPUnit Tests"
         run: vendor/bin/phpunit --configuration phpunit.xml.dist --coverage-text

.github/workflows/docs.yml

@@ -1,13 +1,18 @@
-name: CI
+name: docs
 
 on:
-    push: ~
-    pull_request: ~
+    pull_request:
+        branches:
+            - "*.x"
+            - master
+    push:
+        branches:
+            - "*.x"
+            - master
 
 jobs:
     doctor-rst:
         name: Lint (DOCtor-RST)
-
         runs-on: ubuntu-latest
 
         steps:

.github/workflows/static-analysis.yml

@@ -0,0 +1,42 @@
+name: "static-analysis"
+
+on:
+  pull_request:
+    branches:
+      - "*.x"
+      - master
+  push:
+    branches:
+      - "*.x"
+      - master
+
+jobs:
+  static-analysis:
+    name: "PHPStan"
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: "Checkout"
+        uses: actions/checkout@v4
+        with:
+          fetch-depth: 2
+
+      - name: "Install PHP without coverage"
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: '8.1'
+
+      - name: "Cache dependencies"
+        uses: actions/cache@v3
+        with:
+          path: ${{ steps.composercache.outputs.dir }}
+          key: ${{ runner.os }}-composer-${{ hashFiles('**/composer.json') }}
+          restore-keys: ${{ runner.os }}-composer-
+
+      - name: "Install dependencies with composer"
+        uses: "ramsey/composer-install@v2"
+        with:
+          dependency-versions: 'highest'
+
+      - name: "PHPStan"
+        run: composer phpstan

Tests/Functional/Configs/annotations.yaml

@@ -0,0 +1,4 @@
+framework:
+  annotations: true
+  serializer:
+    enable_annotations: true

Tests/Functional/Controller/MapQueryStringController.php

@@ -10,33 +10,34 @@
 use Nelmio\ApiDocBundle\Tests\Functional\Entity\QueryModel\SortQueryModel;
 use Nelmio\ApiDocBundle\Tests\Functional\Entity\SymfonyConstraintsWithValidationGroups;
 use Nelmio\ApiDocBundle\Tests\Functional\Entity\SymfonyMapQueryString;
+use OpenApi\Attributes as OA;
 use Symfony\Component\HttpKernel\Attribute\MapQueryString;
 use Symfony\Component\Routing\Annotation\Route;
 
 class MapQueryStringController
 {
-    #[Route('/article_map_query_string')]
+    #[Route('/article_map_query_string', methods: ['GET'])]
     #[OA\Response(response: '200', description: '')]
     public function fetchArticleFromMapQueryString(
         #[MapQueryString] SymfonyMapQueryString $article81Query
     ) {
     }
 
-    #[Route('/article_map_query_string_nullable')]
+    #[Route('/article_map_query_string_nullable', methods: ['GET'])]
     #[OA\Response(response: '200', description: '')]
     public function fetchArticleFromMapQueryStringNullable(
         #[MapQueryString] ?SymfonyMapQueryString $article81Query
     ) {
     }
 
-    #[Route('/article_map_query_string_passes_validation_groups')]
+    #[Route('/article_map_query_string_passes_validation_groups', methods: ['GET'])]
     #[OA\Response(response: '200', description: '')]
     public function fetchArticleFromMapQueryStringHandlesValidationGroups(
         #[MapQueryString(validationGroups: ['test'])] SymfonyConstraintsWithValidationGroups $symfonyConstraintsWithValidationGroups,
     ) {
     }
 
-    #[Route('/article_map_query_string_overwrite_parameters')]
+    #[Route('/article_map_query_string_overwrite_parameters', methods: ['GET'])]
     #[OA\Parameter(
         name: 'id',
         in: 'query',
@@ -69,7 +70,7 @@ public function fetchArticleFromMapQueryStringOverwriteParameters(
     ) {
     }
 
-    #[Route('/article_map_query_string_many_parameters')]
+    #[Route('/article_map_query_string_many_parameters', methods: ['GET'])]
     #[OA\Response(response: '200', description: '')]
     public function fetchArticleWithManyParameters(
         #[MapQueryString] FilterQueryModel $filterQuery,
@@ -79,7 +80,7 @@ public function fetchArticleWithManyParameters(
     ) {
     }
 
-    #[Route('/article_map_query_string_many_parameters_optional')]
+    #[Route('/article_map_query_string_many_parameters_optional', methods: ['GET'])]
     #[OA\Response(response: '200', description: '')]
     public function fetchArticleWithManyOptionalParameters(
         #[MapQueryString] ?FilterQueryModel $filterQuery,