Feat: Remove RequireTrailingCommaInCallSniff which is redundant thank…

…s to TrailingCommaInMultilineFixer
lmc-eu · May 23, 2024 · 9dfab40 · 9dfab40
1 parent 4ca51dd
commit 9dfab40
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 4 deletions.
diff --git a/composer.json b/composer.json
@@ -14,7 +14,7 @@
         "friendsofphp/php-cs-fixer": "^3.47.1",
         "slevomat/coding-standard": "^8.6",
         "squizlabs/php_codesniffer": "^3.9",
-        "symplify/easy-coding-standard": "^12.1.9"
+        "symplify/easy-coding-standard": "^12.2.0"
     },
     "require-dev": {
         "ergebnis/composer-normalize": "^2.42.0",

diff --git a/ecs.php b/ecs.php
@@ -162,7 +162,6 @@
 use SlevomatCodingStandard\Sniffs\Classes\RequireConstructorPropertyPromotionSniff;
 use SlevomatCodingStandard\Sniffs\ControlStructures\RequireNullSafeObjectOperatorSniff;
 use SlevomatCodingStandard\Sniffs\Exceptions\ReferenceThrowableOnlySniff;
-use SlevomatCodingStandard\Sniffs\Functions\RequireTrailingCommaInCallSniff;
 use SlevomatCodingStandard\Sniffs\TypeHints\ParameterTypeHintSniff;
 use SlevomatCodingStandard\Sniffs\TypeHints\PropertyTypeHintSniff;
 use SlevomatCodingStandard\Sniffs\TypeHints\ReturnTypeHintSniff;
@@ -445,8 +444,6 @@
             // Promote constructor properties
             // For php-cs-fixer implementation @see https://github.com/FriendsOfPHP/PHP-CS-Fixer/issues/5956
             RequireConstructorPropertyPromotionSniff::class,
-            // Multi-line arguments list in function/method call must have a trailing comma
-            RequireTrailingCommaInCallSniff::class, // TODO: will be redundant after https://github.com/PHP-CS-Fixer/PHP-CS-Fixer/pull/7989 is merged and released
             // Use `null-safe` operator `?->` where possible
             RequireNullSafeObjectOperatorSniff::class,
         ],