Merge pull request #94 from sasezaki/reason-tip

Refactor to use acceptsReasonsTip() for consistent error message handling

Author: sasezaki

src/Rules/ContextTypeRule.php

@@ -17,7 +17,6 @@
 use Sfp\PHPStan\Psr\Log\TypeProviderResolver\ContextTypeProviderResolverInterface;
 
 use function count;
-use function implode;
 use function in_array;
 use function sprintf;
 
@@ -99,12 +98,9 @@ public function processNode(Node $node, Scope $scope): array
                 (string) $argContextType->toPhpDocNode()
             )
         )
+            ->acceptsReasonsTip($acceptsResult->reasons)
             ->identifier('sfpPsrLog.contextType');
 
-        if (count($acceptsResult->reasons) > 0) {
-            $ruleErrorBuilder->tip(implode(',', $acceptsResult->reasons));
-        }
-
         return [
             $ruleErrorBuilder->build(),
         ];

src/Rules/LogMethodLevelRule.php

@@ -96,7 +96,10 @@ public function processNode(Node $node, Scope $scope): array
                     $this->acceptingLogLevel->toPhpDocNode()->__toString(),
                     $argLevel->toPhpDocNode()->__toString()
                 )
-            )->identifier('sfpPsrLog.logMethodLevel')->build(),
+            )
+                ->acceptsReasonsTip($acceptsResult->reasons)
+                ->identifier('sfpPsrLog.logMethodLevel')
+                ->build(),
         ];
     }
 }