Update "trailing_comma_in_multiline" to add comma to arguments and pa…

…rameters (#21)

.php-cs-fixer.php

@@ -16,5 +16,5 @@
             ->files()
             ->in(__DIR__ . '/src')
             ->in(__DIR__ . '/tests')
-            ->append([__FILE__])
+            ->append([__FILE__]),
     );

src/Builder/Builder.php

@@ -61,7 +61,7 @@ private function dumpClass(string $name, Rules $rules): void
             $content = \str_replace(
                 \sprintf("'%s'", $rule),
                 \sprintf('Fixer\\%sFixer::name()', \ucfirst(\str_replace([' ', 'PhpCsFixerCustomFixers/'], '', \ucwords(\str_replace('_', ' ', $rule))))),
-                $content
+                $content,
             );
         }
 

src/Builder/Modifier/NonDefaultConfiguration.php

@@ -38,6 +38,7 @@ public function __invoke(array $rules): array
         $rules['no_superfluous_phpdoc_tags'] = ['remove_inheritdoc' => true];
         $rules['php_unit_test_case_static_method_calls'] = ['call_type' => 'self'];
         $rules['phpdoc_line_span'] = ['property' => 'single'];
+        $rules['trailing_comma_in_multiline'] = ['after_heredoc' => true, 'elements' => ['arguments', 'arrays', 'parameters']];
         $rules['yoda_style'] = ['equal' => false, 'identical' => false, 'less_and_greater' => false];
         $rules[NoSuperfluousConcatenationFixer::name()] = ['allow_preventing_trailing_spaces' => true];
 

src/Rules/LibraryRules.php

@@ -34,7 +34,7 @@ public function __construct(string $name, string $author, int $year)
         ',
             $name,
             $year,
-            $author
+            $author,
         ));
     }
 
@@ -323,7 +323,14 @@ public function getRules(): array
             'ternary_operator_spaces' => true,
             'ternary_to_elvis_operator' => true,
             'ternary_to_null_coalescing' => true,
-            'trailing_comma_in_multiline' => true,
+            'trailing_comma_in_multiline' => [
+                'after_heredoc' => true,
+                'elements' => [
+                    'arguments',
+                    'arrays',
+                    'parameters',
+                ],
+            ],
             'trim_array_spaces' => true,
             'types_spaces' => true,
             'unary_operator_spaces' => true,

src/Rules/ProjectRules.php

@@ -300,7 +300,14 @@ public function getRules(): array
             'ternary_operator_spaces' => true,
             'ternary_to_elvis_operator' => true,
             'ternary_to_null_coalescing' => true,
-            'trailing_comma_in_multiline' => true,
+            'trailing_comma_in_multiline' => [
+                'after_heredoc' => true,
+                'elements' => [
+                    'arguments',
+                    'arrays',
+                    'parameters',
+                ],
+            ],
             'trim_array_spaces' => true,
             'types_spaces' => true,
             'unary_operator_spaces' => true,

tests/AutoReview/SrcCodeTest.php

@@ -46,7 +46,7 @@ public function testOnlyFactoryClassIsPublic(): void
             self::assertSame(
                 $class === Factory::class,
                 \mb_strpos($reflectionClass->getDocComment(), '@internal') === false,
-                \sprintf('Class %s has "@internal" tag incorrectly.', $class)
+                \sprintf('Class %s has "@internal" tag incorrectly.', $class),
             );
         }
     }

tests/Builder/Modifier/NonDefaultConfigurationTest.php

@@ -42,7 +42,7 @@ static function (string $x, string $y): int {
                 }
 
                 return $x <=> $y;
-            }
+            },
         );
 
         self::assertSame($sortedRules, $rules);

tests/FactoryTest.php

@@ -26,15 +26,15 @@ public function testCreateForLibrary(): void
     {
         self::assertInstanceOf(
             Config::class,
-            Factory::createForLibrary('library name', 'library author', 2000)
+            Factory::createForLibrary('library name', 'library author', 2000),
         );
     }
 
     public function testCreateForProject(): void
     {
         self::assertInstanceOf(
             Config::class,
-            Factory::createForProject()
+            Factory::createForProject(),
         );
     }
 }

tests/Rules/LibraryRulesTest.php

@@ -47,7 +47,7 @@ public function testRulesAreUpToDate(): void
 
         self::assertSame(
             $rules->getRules(),
-            $libraryRules->getRules()
+            $libraryRules->getRules(),
         );
     }
 
@@ -63,7 +63,7 @@ public function testRulesBuildInConfig(): void
 
         self::assertInstanceOf(
             FixerFactory::class,
-            $fixerFactory->useRuleSet($ruleSet)
+            $fixerFactory->useRuleSet($ruleSet),
         );
     }
 }

tests/Rules/ProjectRulesTest.php

@@ -35,7 +35,7 @@ public function testRulesAreUpToDate(): void
 
         self::assertSame(
             $rules->getRules(),
-            $projectRules->getRules()
+            $projectRules->getRules(),
         );
     }
 
@@ -51,7 +51,7 @@ public function testRulesBuildInConfig(): void
 
         self::assertInstanceOf(
             FixerFactory::class,
-            $fixerFactory->useRuleSet($ruleSet)
+            $fixerFactory->useRuleSet($ruleSet),
         );
     }
 }