minor #54 Group some PHPDoc tags in PhpdocSeparationFixer rule (coldic3)

This PR was merged into the 4.3-dev branch.

Discussion
----------



Commits
-------

8163ed2 Group some PHPDoc tags in PhpdocSeparationFixer rule

ecs.php

@@ -39,6 +39,7 @@
 use PhpCsFixer\Fixer\Comment\NoEmptyCommentFixer;
 use PhpCsFixer\Fixer\Comment\NoTrailingWhitespaceInCommentFixer;
 use PhpCsFixer\Fixer\Comment\SingleLineCommentStyleFixer;
+use PhpCsFixer\Fixer\ConfigurableFixerInterface;
 use PhpCsFixer\Fixer\ConstantNotation\NativeConstantInvocationFixer;
 use PhpCsFixer\Fixer\ControlStructure\ElseifFixer;
 use PhpCsFixer\Fixer\ControlStructure\IncludeFixer;
@@ -257,5 +258,10 @@
     $ecsConfig->ruleWithConfiguration(TrailingCommaInMultilineFixer::class, ['elements' => ['arrays', 'arguments', 'parameters']]);
     $ecsConfig->ruleWithConfiguration(VisibilityRequiredFixer::class, ['elements' => ['const', 'property', 'method']]);
 
+    // ECS 10 issue
+    if (is_a(PhpdocSeparationFixer::class, ConfigurableFixerInterface::class, true)) {
+        $ecsConfig->ruleWithConfiguration(PhpdocSeparationFixer::class, ['groups' => [['Given', 'When', 'Then'], ['var', 'phpstan-var', 'psalm-var'], ['return', 'phpstan-return', 'psalm-return'], ['param', 'phpstan-param', 'psalm-param']]]);
+    }
+
     $ecsConfig->skip([VisibilityRequiredFixer::class => ['*Spec.php']]);
 };

tests/Annotations.php

@@ -0,0 +1,32 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Tests\CodingStandard;
+
+final class Annotations
+{
+    /**
+     * @param int $bar
+     * @param string $baz
+     * @phpstan-param string $baz
+     * @psalm-param string $baz
+     *
+     * @return string[]
+     * @phpstan-return string<int, string>
+     * @psalm-return string<int, string>
+     */
+    public function foo($bar, $baz)
+    {
+        /**
+         * Some comment
+         *
+         * @var int $key
+         * @phpstan-var int $key
+         * @psalm-var int $key
+         */
+        $key = --$bar;
+
+        return [$key => $baz];
+    }
+}