Merge pull request #579 from ergebnis/feature/no-trailing-comma-in-si…

…ngleline-function-call Enhancement: Enable `no_trailing_comma_in_singleline_function_call` fixer
ergebnis · Mar 7, 2022 · caef33f · caef33f
2 parents 74edb44 + 673cc29
commit caef33f
Show file tree

Hide file tree

Showing 7 changed files with 8 additions and 6 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -11,6 +11,7 @@ For a full diff see [`4.1.0...main`][4.1.0...main].
 ### Changed
 
 - Updated `friendsofphp/php-cs-fixer` ([#578]), by [@dependabot]
+- Enabled `no_trailing_comma_in_singleline_function_call` fixer ([#579]), by [@localheinz]
 
 ## [`4.1.0`][4.1.0]
 
@@ -588,6 +589,7 @@ For a full diff see [`d899e77...1.0.0`][d899e77...1.0.0].
 [#566]: https://github.com/ergebnis/php-cs-fixer-config/pull/566
 [#567]: https://github.com/ergebnis/php-cs-fixer-config/pull/567
 [#578]: https://github.com/ergebnis/php-cs-fixer-config/pull/578
+[#579]: https://github.com/ergebnis/php-cs-fixer-config/pull/579
 
 [@dependabot]: https://github.com/apps/dependabot
 [@linuxjuggler]: https://github.com/linuxjuggler

diff --git a/src/RuleSet/Php74.php b/src/RuleSet/Php74.php
@@ -325,7 +325,7 @@ final class Php74 extends AbstractRuleSet implements ExplicitRuleSet
         ],
         'no_trailing_comma_in_list_call' => true,
         'no_trailing_comma_in_singleline_array' => true,
-        'no_trailing_comma_in_singleline_function_call' => false,
+        'no_trailing_comma_in_singleline_function_call' => true,
         'no_trailing_whitespace' => true,
         'no_trailing_whitespace_in_comment' => true,
         'no_trailing_whitespace_in_string' => true,

diff --git a/src/RuleSet/Php80.php b/src/RuleSet/Php80.php
@@ -325,7 +325,7 @@ final class Php80 extends AbstractRuleSet implements ExplicitRuleSet
         ],
         'no_trailing_comma_in_list_call' => true,
         'no_trailing_comma_in_singleline_array' => true,
-        'no_trailing_comma_in_singleline_function_call' => false,
+        'no_trailing_comma_in_singleline_function_call' => true,
         'no_trailing_whitespace' => true,
         'no_trailing_whitespace_in_comment' => true,
         'no_trailing_whitespace_in_string' => true,

diff --git a/src/RuleSet/Php81.php b/src/RuleSet/Php81.php
@@ -325,7 +325,7 @@ final class Php81 extends AbstractRuleSet implements ExplicitRuleSet
         ],
         'no_trailing_comma_in_list_call' => true,
         'no_trailing_comma_in_singleline_array' => true,
-        'no_trailing_comma_in_singleline_function_call' => false,
+        'no_trailing_comma_in_singleline_function_call' => true,
         'no_trailing_whitespace' => true,
         'no_trailing_whitespace_in_comment' => true,
         'no_trailing_whitespace_in_string' => true,

diff --git a/test/Unit/RuleSet/Php74Test.php b/test/Unit/RuleSet/Php74Test.php
@@ -331,7 +331,7 @@ final class Php74Test extends ExplicitRuleSetTestCase
         ],
         'no_trailing_comma_in_list_call' => true,
         'no_trailing_comma_in_singleline_array' => true,
-        'no_trailing_comma_in_singleline_function_call' => false,
+        'no_trailing_comma_in_singleline_function_call' => true,
         'no_trailing_whitespace' => true,
         'no_trailing_whitespace_in_comment' => true,
         'no_trailing_whitespace_in_string' => true,

diff --git a/test/Unit/RuleSet/Php80Test.php b/test/Unit/RuleSet/Php80Test.php
@@ -331,7 +331,7 @@ final class Php80Test extends ExplicitRuleSetTestCase
         ],
         'no_trailing_comma_in_list_call' => true,
         'no_trailing_comma_in_singleline_array' => true,
-        'no_trailing_comma_in_singleline_function_call' => false,
+        'no_trailing_comma_in_singleline_function_call' => true,
         'no_trailing_whitespace' => true,
         'no_trailing_whitespace_in_comment' => true,
         'no_trailing_whitespace_in_string' => true,

diff --git a/test/Unit/RuleSet/Php81Test.php b/test/Unit/RuleSet/Php81Test.php
@@ -331,7 +331,7 @@ final class Php81Test extends ExplicitRuleSetTestCase
         ],
         'no_trailing_comma_in_list_call' => true,
         'no_trailing_comma_in_singleline_array' => true,
-        'no_trailing_comma_in_singleline_function_call' => false,
+        'no_trailing_comma_in_singleline_function_call' => true,
         'no_trailing_whitespace' => true,
         'no_trailing_whitespace_in_comment' => true,
         'no_trailing_whitespace_in_string' => true,