Merge branch 'master' into PSR12

PHP-CS-Fixer · Oct 25, 2023 · f5614f6 · f5614f6
2 parents abddff1 + e008ba6
commit f5614f6
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 6 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -34,11 +34,11 @@ jobs:
 
           - operating-system: 'ubuntu-20.04'
             php-version: '8.0'
-            job-description: 'with Symfony ^6'
-            execute-flex-with-symfony-version: '^6' # explicit check for Symfony 6.x compatibility
 
           - operating-system: 'ubuntu-20.04'
             php-version: '8.1'
+            job-description: 'with Symfony ^6'
+            execute-flex-with-symfony-version: '^6' # explicit check for Symfony 6.x compatibility
 
           - operating-system: 'ubuntu-20.04'
             php-version: '8.2'

diff --git a/doc/rules/language_construct/nullable_type_declaration.rst b/doc/rules/language_construct/nullable_type_declaration.rst
@@ -30,8 +30,8 @@ Example #1
    --- Original
    +++ New
     <?php
-   -function bar(null|int $value, null|\Closure $callable): void {}
-   +function bar(?int $value, ?\Closure $callable): void {}
+   -function bar(null|int $value, null|\Closure $callable): int|null {}
+   +function bar(?int $value, ?\Closure $callable): ?int {}
 
 Example #2
 ~~~~~~~~~~

diff --git a/src/Fixer/LanguageConstruct/NullableTypeDeclarationFixer.php b/src/Fixer/LanguageConstruct/NullableTypeDeclarationFixer.php
@@ -47,7 +47,7 @@ public function getDefinition(): FixerDefinitionInterface
             'Nullable single type declaration should be standardised using configured syntax.',
             [
                 new VersionSpecificCodeSample(
-                    "<?php\nfunction bar(null|int \$value, null|\\Closure \$callable): void {}\n",
+                    "<?php\nfunction bar(null|int \$value, null|\\Closure \$callable): int|null {}\n",
                     new VersionSpecification(8_00_00)
                 ),
                 new VersionSpecificCodeSample(

diff --git a/tests/AutoReview/FixerFactoryTest.php b/tests/AutoReview/FixerFactoryTest.php
@@ -323,7 +323,7 @@ public function testFixerWithNoneDefaultPriorityIsTested(): void
             }
         }
 
-        self::assertEmpty($missing, 'Fixers without default priority and without priority tests: "'.implode('", "', array_keys($missing)).'."');
+        self::assertEmpty($missing, 'Fixers with non-default priority and yet without priority unit tests [vide "getFixersPriorityGraph()" and "getPhpDocFixersPriorityGraph()"]: "'.implode('", "', array_keys($missing)).'."');
     }
 
     /**