Adds support for GrumPHP and updates script installation commands in composer.json

Author: coisa

.docheader

@@ -10,4 +10,5 @@
  * @see       https://github.com/php-fast-forward/dev-tools
  * @see       https://github.com/php-fast-forward
  * @see       https://datatracker.ietf.org/doc/html/rfc2119
- */
+ */
+ 

bin/dev-tools.php

@@ -16,12 +16,12 @@
  * @see       https://datatracker.ietf.org/doc/html/rfc2119
  */
 
-$projectVendorAutoload = dirname(__DIR__, 4) . '/vendor/autoload.php';
-$pluginVendorAutoload = dirname(__DIR__) . '/vendor/autoload.php';
+namespace FastForward\DevTools;
 
-require_once file_exists($projectVendorAutoload) ? $projectVendorAutoload : $pluginVendorAutoload;
+$projectVendorAutoload = \dirname(__DIR__, 4) . '/vendor/autoload.php';
+$pluginVendorAutoload = \dirname(__DIR__) . '/vendor/autoload.php';
 
-use FastForward\DevTools\DevTools;
+require_once file_exists($projectVendorAutoload) ? $projectVendorAutoload : $pluginVendorAutoload;
 
 $application = new DevTools();
 $application->run();

composer.json

@@ -76,6 +76,6 @@
     },
     "scripts": {
         "dev-tools": "dev-tools",
-        "dev-tools:fix": "dev-tools --fix"
+        "dev-tools:fix": "@dev-tools --fix"
     }
 }

src/Command/InstallScriptsCommand.php

@@ -72,13 +72,13 @@ protected function execute(InputInterface $input, OutputInterface $output): int
 
         $scripts = [
             'dev-tools' => 'dev-tools',
-            'dev-tools:fix' => 'dev-tools --fix',
+            'dev-tools:fix' => '@dev-tools --fix',
         ];
 
         $extra = [
             'grumphp' => [
                 'config-default-path' => Path::makeRelative(
-                    dirname(__DIR__, 2) . '/grumphp.yml',
+                    \dirname(__DIR__, 2) . '/grumphp.yml',
                     $this->getCurrentWorkingDirectory(),
                 ),
             ],

src/Command/ReportsCommand.php

@@ -21,7 +21,6 @@
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
 
-
 /**
  * Coordinates the generation of Fast Forward documentation frontpage and related reports.
  * This class MUST NOT be overridden and SHALL securely combine docs and testing commands.
@@ -77,6 +76,6 @@ protected function execute(InputInterface $input, OutputInterface $output): int
 
         $output->writeln('<info>Frontpage generation completed!</info>');
 
-        return in_array(self::FAILURE, $results, true) ? self::FAILURE : self::SUCCESS;
+        return \in_array(self::FAILURE, $results, true) ? self::FAILURE : self::SUCCESS;
     }
 }

src/Command/StandardsCommand.php

@@ -77,6 +77,6 @@ protected function execute(InputInterface $input, OutputInterface $output): int
 
         $output->writeln('<info>All code standards checks completed!</info>');
 
-        return in_array(self::FAILURE, $results, true) ? self::FAILURE : self::SUCCESS;
+        return \in_array(self::FAILURE, $results, true) ? self::FAILURE : self::SUCCESS;
     }
 }

src/Composer/Plugin.php

@@ -77,7 +77,9 @@ public static function getSubscribedEvents(): array
      */
     public function onPostInstall(Event $event): void
     {
-        $event->getComposer()->getEventDispatcher()->dispatchScript('install-scripts', true);
+        $event->getComposer()
+            ->getEventDispatcher()
+            ->dispatchScript('install-scripts', true);
     }
 
     /**
@@ -92,7 +94,9 @@ public function onPostInstall(Event $event): void
      */
     public function onPostUpdate(Event $event): void
     {
-        $event->getComposer()->getEventDispatcher()->dispatchScript('install-scripts', true);
+        $event->getComposer()
+            ->getEventDispatcher()
+            ->dispatchScript('install-scripts', true);
     }
 
     /**

src/Rector/AddMissingClassPhpDocRector.php

@@ -41,10 +41,7 @@ final class AddMissingClassPhpDocRector extends AbstractRector
     public function getRuleDefinition(): RuleDefinition
     {
         return new RuleDefinition('Add basic PHPDoc to classes without docblock', [
-            new CodeSample(
-                'class SomeClass {}',
-                "/**\n * SomeClass\n */\nclass SomeClass {}"
-            ),
+            new CodeSample('class SomeClass {}', "/**\n * SomeClass\n */\nclass SomeClass {}"),
         ]);
     }
 

src/Rector/AddMissingMethodPhpDocRector.php

@@ -59,10 +59,7 @@ final class AddMissingMethodPhpDocRector extends AbstractRector
     public function getRuleDefinition(): RuleDefinition
     {
         return new RuleDefinition('Add basic PHPDoc to methods without docblock', [
-            new CodeSample(
-                'public function foo() {}',
-                "/**\n * \n */\npublic function foo() {}"
-            ),
+            new CodeSample('public function foo() {}', "/**\n * \n */\npublic function foo() {}"),
         ]);
     }
 

src/Rector/RemoveEmptyDocBlockRector.php

@@ -45,10 +45,7 @@ final class RemoveEmptyDocBlockRector extends AbstractRector
     public function getRuleDefinition(): RuleDefinition
     {
         return new RuleDefinition('Remove empty docblocks from classes and methods', [
-            new CodeSample(
-                "/**\n *\n */\nclass SomeClass {}",
-                'class SomeClass {}'
-            ),
+            new CodeSample("/**\n *\n */\nclass SomeClass {}", 'class SomeClass {}'),
         ]);
     }
 
@@ -130,9 +127,11 @@ private function isEmptyDocBlock(string $docBlock): bool
             if ('/**' === $normalizedLine) {
                 continue;
             }
+
             if ('*/' === $normalizedLine) {
                 continue;
             }
+
             if ('*' === $normalizedLine) {
                 continue;
             }