clean other directives

Author: nikophil

_build/src/Directive/AbstractAdmonitionDirective.php

@@ -21,13 +21,8 @@ public function __construct(string $name, string $text)
         $this->text = $text;
     }
 
-    final public function processSub(
-        Parser $parser,
-        ?Node $document,
-        string $variable,
-        string $data,
-        array $options
-    ): ?Node {
+    final public function processSub(Parser $parser, ?Node $document, string $variable, string $data, array $options): ?Node
+    {
         return new WrapperNode(
             $document,
             sprintf(

_build/src/Directive/ClassDirective.php

@@ -15,15 +15,16 @@ public function getName(): string
         return 'class';
     }
 
-    public function processSub(
-        Parser $parser,
-        ?Node $document,
-        string $variable,
-        string $data,
-        array $options
-    ): ?Node {
+    public function processSub(Parser $parser, ?Node $document, string $variable, string $data, array $options): ?Node
+    {
         if (!$document instanceof ListNode && !$document instanceof ParagraphNode) {
-            throw new \RuntimeException('Class could only be applied to paragraphs or lists');
+            throw new \RuntimeException(
+                sprintf(
+                    "\".. class:\" directive could only be applied to paragraphs or lists (applied to \"%s\")\nTarget block:\n%s",
+                    \get_class($document),
+                    $document->render()
+                )
+            );
         }
 
         $document->setClass($data);

_build/src/Directive/CodeBlockDirective.php

@@ -6,46 +6,23 @@
 use Doctrine\RST\Nodes\CodeNode;
 use Doctrine\RST\Nodes\Node;
 use Doctrine\RST\Parser;
-use function array_reverse;
-use function assert;
-use function is_array;
-use function is_string;
-use function preg_split;
-use function trim;
 
-/**
- * Renders a code block, example:
- *
- * .. code-block:: php
- *
- *      <?php
- *
- *      echo "Hello world!\n";
- */
 class CodeBlockDirective extends Directive
 {
     public function getName(): string
     {
         return 'code-block';
     }
 
-    /**
-     * @param string[] $options
-     */
-    public function process(
-        Parser $parser,
-        ?Node $node,
-        string $variable,
-        string $data,
-        array $options
-    ): void {
+    public function process(Parser $parser, ?Node $node, string $variable, string $data, array $options): void
+    {
         if (!$node instanceof CodeNode) {
             return;
         }
 
         $node->setLanguage($data);
 
-        if ($variable !== '') {
+        if ('' !== $variable) {
             $environment = $parser->getEnvironment();
             $environment->setVariable($variable, $node);
         } else {

_build/src/Directive/ConfigurationBlockDirective.php

@@ -17,16 +17,8 @@ public function getName(): string
         return 'configuration-block';
     }
 
-    /**
-     * @param string[] $options
-     */
-    public function processSub(
-        Parser $parser,
-        ?Node $document,
-        string $variable,
-        string $data,
-        array $options
-    ): ?Node {
+    public function processSub(Parser $parser, ?Node $document, string $variable, string $data, array $options): ?Node
+    {
         if (!$document instanceof Document) {
             return null;
         }

_build/src/Directive/RoleDirective.php

@@ -9,21 +9,13 @@
 
 class RoleDirective extends SubDirective
 {
-    public function getName() : string
+    public function getName(): string
     {
         return 'role';
     }
 
-    /**
-     * @param string[] $options
-     */
-    public function processSub(
-        Parser $parser,
-        ?Node $document,
-        string $variable,
-        string $data,
-        array $options
-    ) : ?Node {
+    public function processSub(Parser $parser, ?Node $document, string $variable, string $data, array $options): ?Node
+    {
         return new WrapperNode($document, '<div class="role">', '</div>');
     }
 }

_build/src/Directive/SidebarDirective.php

@@ -14,16 +14,8 @@ public function getName(): string
         return 'sidebar';
     }
 
-    /**
-     * @param string[] $options
-     */
-    public function processSub(
-        Parser $parser,
-        ?Node $document,
-        string $variable,
-        string $data,
-        array $options
-    ): ?Node {
+    public function processSub(Parser $parser, ?Node $document, string $variable, string $data, array $options): ?Node
+    {
         return new WrapperNode(
             $document,
             sprintf('<div class="admonition-wrapper"><div class="sidebar"></div><div class="admonition admonition-sidebar"><p class="sidebar-title">%s</p>', $data),

_build/src/Directive/VersionAddedDirective.php

@@ -14,16 +14,8 @@ public function getName(): string
         return 'versionadded';
     }
 
-    /**
-     * @param string[] $options
-     */
-    public function processSub(
-        Parser $parser,
-        ?Node $document,
-        string $variable,
-        string $data,
-        array $options
-    ): ?Node {
+    public function processSub(Parser $parser, ?Node $document, string $variable, string $data, array $options): ?Node
+    {
         return new WrapperNode(
             $document,
             sprintf('<div class="versionadded"><div><span class="versionmodified">New in version %s: </span>', $data),