cs: enable RequireMultiLineMethodSignature (#80)

shipmonk-rnd · Feb 2, 2023 · 24142f7 · 24142f7
1 parent bae7d58
commit 24142f7
Show file tree

Hide file tree

Showing 4 changed files with 25 additions and 4 deletions.
diff --git a/phpcs.xml.dist b/phpcs.xml.dist
@@ -270,6 +270,7 @@
     <rule ref="SlevomatCodingStandard.Classes.EmptyLinesAroundClassBraces"/>
     <rule ref="SlevomatCodingStandard.Classes.TraitUseDeclaration" />
     <rule ref="SlevomatCodingStandard.Classes.PropertyDeclaration" />
+    <rule ref="SlevomatCodingStandard.Classes.RequireMultiLineMethodSignature"/>
     <rule ref="SlevomatCodingStandard.Classes.TraitUseSpacing" />
     <rule ref="SlevomatCodingStandard.Classes.DisallowConstructorPropertyPromotion" />
     <rule ref="SlevomatCodingStandard.Commenting.DocCommentSpacing">

diff --git a/src/Rule/EnforceNativeReturnTypehintRule.php b/src/Rule/EnforceNativeReturnTypehintRule.php
@@ -294,7 +294,12 @@ private function getUnionTypehint(Type $type, Scope $scope, bool $typeFromPhpDoc
         return implode('|', $typehintParts);
     }
 
-    private function getIntersectionTypehint(Type $type, Scope $scope, bool $typeFromPhpDoc, bool $alwaysTerminating): ?string
+    private function getIntersectionTypehint(
+        Type $type,
+        Scope $scope,
+        bool $typeFromPhpDoc,
+        bool $alwaysTerminating
+    ): ?string
     {
         if (!$type instanceof IntersectionType) {
             return null;

diff --git a/src/Rule/ForbidPhpDocNullabilityMismatchWithNativeTypehintRule.php b/src/Rule/ForbidPhpDocNullabilityMismatchWithNativeTypehintRule.php
@@ -200,7 +200,12 @@ private function getPhpDocParamType(FunctionLike $node, Scope $scope, string $pa
     /**
      * @return list<string>
      */
-    private function comparePhpDocAndNativeType(?Type $phpDocReturnType, ?Type $nativeReturnType, Scope $scope, string $phpDocIdentification): array
+    private function comparePhpDocAndNativeType(
+        ?Type $phpDocReturnType,
+        ?Type $nativeReturnType,
+        Scope $scope,
+        string $phpDocIdentification
+    ): array
     {
         if ($phpDocReturnType === null || $nativeReturnType === null) {
             return [];

diff --git a/src/Rule/RequirePreviousExceptionPassRule.php b/src/Rule/RequirePreviousExceptionPassRule.php
@@ -97,7 +97,13 @@ public function processNode(Node $node, Scope $scope): array
     /**
      * @return list<RuleError>
      */
-    private function processExceptionCreation(bool $strictTypes, ?string $caughtExceptionVariableName, Type $caughtExceptionType, CallLike $node, Scope $scope): array
+    private function processExceptionCreation(
+        bool $strictTypes,
+        ?string $caughtExceptionVariableName,
+        Type $caughtExceptionType,
+        CallLike $node,
+        Scope $scope
+    ): array
     {
         $passed = false;
 
@@ -170,7 +176,11 @@ private function getCallLikeParameters(CallLike $node, Scope $scope): array
     /**
      * @param Name[] $exceptionNames
      */
-    private function getCaughtExceptionType(array $exceptionNames, Scope $scope, Type $exceptionTypesCaughtInPreviousCatches): Type
+    private function getCaughtExceptionType(
+        array $exceptionNames,
+        Scope $scope,
+        Type $exceptionTypesCaughtInPreviousCatches
+    ): Type
     {
         $classes = [];