ruleset interface improvements

Author: b-vadym

psalm.xml

@@ -13,4 +13,11 @@
             <directory name="vendor" />
         </ignoreFiles>
     </projectFiles>
+    <issueHandlers>
+        <MixedReturnTypeCoercion>
+            <errorLevel type="suppress">
+                <directory name="src"/>
+            </errorLevel>
+        </MixedReturnTypeCoercion>
+    </issueHandlers>
 </psalm>

src/Factory.php

@@ -16,13 +16,12 @@ public static function fromRuleSet(RuleSetInterface $ruleSet, array $overrideRul
         $config = new Config($ruleSet->name());
 
         $config
-            ->setRiskyAllowed(true)
+            ->setRiskyAllowed($ruleSet->getRiskyAllowed())
             ->setRules(array_merge(
                 $ruleSet->rules(),
                 $overrideRules
             ))
-            ->registerCustomFixers(new \PhpCsFixerCustomFixers\Fixers())
-            ->registerCustomFixers(new \PedroTroller\CS\Fixer\Fixers())
+            ->registerCustomFixers($ruleSet->getCustomFixers())
         ;
 
         return $config;

src/RuleSet/Php74.php

@@ -97,4 +97,15 @@ public function rules(): array
             'PedroTroller/line_break_between_statements' => true,
         ];
     }
+
+    public function getCustomFixers(): iterable
+    {
+        yield from new \PhpCsFixerCustomFixers\Fixers();
+        yield from new \PedroTroller\CS\Fixer\Fixers();
+    }
+
+    public function getRiskyAllowed(): bool
+    {
+        return true;
+    }
 }

src/RuleSet/Php80.php

@@ -97,4 +97,15 @@ public function rules(): array
             'PedroTroller/line_break_between_statements' => true,
         ];
     }
+
+    public function getCustomFixers(): iterable
+    {
+        yield from new \PhpCsFixerCustomFixers\Fixers();
+        yield from new \PedroTroller\CS\Fixer\Fixers();
+    }
+
+    public function getRiskyAllowed(): bool
+    {
+        return true;
+    }
 }

src/RuleSetInterface.php

@@ -4,6 +4,8 @@
 
 namespace VisualCraft\PhpCsFixerConfig;
 
+use PhpCsFixer\Fixer\FixerInterface;
+
 interface RuleSetInterface
 {
     /**
@@ -17,4 +19,11 @@ public function name(): string;
      * @psalm-return array<string, array|bool>
      */
     public function rules(): array;
+
+    /**
+     * @return iterable<array-key, FixerInterface>
+     */
+    public function getCustomFixers(): iterable;
+
+    public function getRiskyAllowed(): bool;
 }