anroots / pgca

src/Anroots/Pgca/AbstractSetConfigurator.php

Doc Comments +1 added lines

@@ -85,6 +85,7 @@
 
     /**
      * {@inheritdoc}
+     * @param string $suffix
      */
     protected function getServiceName($suffix)
     {

Spacing +2 added lines, -2 removed lines

@@ -78,7 +78,7 @@ 
             throw new ServiceNotFoundException($this->getServiceName($serviceName));
         }
 
-        $service = $this->container->get($this->prefix . $serviceName);
+        $service = $this->container->get($this->prefix.$serviceName);
 
         return $service;
     }
@@ -88,7 +88,7 @@ 
      */
     protected function getServiceName($suffix)
     {
-        return $this->prefix . $suffix;
+        return $this->prefix.$suffix;
     }
 
     /**

src/Anroots/Pgca/Report/Serializer/ConsoleSerializer.php

Doc Comments +1 added lines, -1 removed lines

@@ -96,7 +96,7 @@
     }
 
     /**
-     * @param array $rows
+     * @param ReportColumn[] $rows
      */
     private function printBody(array $rows)
     {

Spacing +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
     private function getHeader(ReportComposerInterface $report)
     {
         // Todo: write a pretty and more informational header
-        return 'PGCA report, generated on ' . $report->getReportHeader()->getCreated()->format('Y-m-d H:i:s');
+        return 'PGCA report, generated on '.$report->getReportHeader()->getCreated()->format('Y-m-d H:i:s');
     }
 
     /**

src/Anroots/Pgca/Cli/Command/Rules/ShowCommand.php

Spacing +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
 
         try {
             /** @var RuleInterface $rule */
-            $rule = $this->getContainer()->get('rule.' . $input->getArgument('name'));
+            $rule = $this->getContainer()->get('rule.'.$input->getArgument('name'));
         } catch (InvalidArgumentException $e) {
             $output->writeln(sprintf('<error>The rule "%s" does not exist.</error>', $input->getArgument('name')));
 

src/Anroots/Pgca/Cli/Command/AbstractAnalyzeCommand.php

Spacing +4 added lines, -4 removed lines

@@ -94,7 +94,7 @@ 
      */
     private function providerFactory(array $providerConfig)
     {
-        $providerServiceName = 'commit.provider.' . $providerConfig['name'];
+        $providerServiceName = 'commit.provider.'.$providerConfig['name'];
 
         /** @var CommitProviderInterface $provider */
         $provider = $this->getContainer()->get($providerServiceName);
@@ -109,7 +109,7 @@ 
      */
     private function reportComposerFactory($name)
     {
-        return $this->getContainer()->get('report.composer.' . $name . 'Report');
+        return $this->getContainer()->get('report.composer.'.$name.'Report');
     }
 
     /**
@@ -118,7 +118,7 @@ 
      */
     private function serializerFactory($serializer)
     {
-        return $this->getContainer()->get('report.serializer.' . $serializer . 'Serializer');
+        return $this->getContainer()->get('report.serializer.'.$serializer.'Serializer');
     }
 
     /**
@@ -127,7 +127,7 @@ 
      */
     private function printerFactory($name)
     {
-        $printer = $this->getContainer()->get('report.printer.' . $name . 'Printer');
+        $printer = $this->getContainer()->get('report.printer.'.$name.'Printer');
 
         // Todo: refactor with the correct design pattern. Lose the if-s
         if ($printer instanceof ConsolePrinter) {

src/Anroots/Pgca/Cli/Application.php

Spacing +2 added lines, -2 removed lines

@@ -55,8 +55,8 @@
     {
         $container = new ContainerBuilder;
         $fileLocator = new FileLocator([
-            __DIR__ . '/../../../../config',
-            __DIR__ . '/../../../../../../../config'
+            __DIR__.'/../../../../config',
+            __DIR__.'/../../../../../../../config'
         ]);
         $loader = new XmlFileLoader($container, $fileLocator);
         $loader->load('services.xml');

src/Anroots/Pgca/Report/Serializer/HtmlSerializer.php

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
         $rows = $this->getRows($report->getReportHeader()->getRows());
 
         $html = $this->engine->loadTemplate(
-            file_get_contents(__DIR__ . '/../../../../../assets/templates/report.mustache')
+            file_get_contents(__DIR__.'/../../../../../assets/templates/report.mustache')
         )
             ->render(['rows' => $rows]);
 

src/Anroots/Pgca/Report/Composer/SimpleReport.php

Spacing +1 added lines, -1 removed lines

@@ -42,6 +42,6 @@
             return $text;
         }
 
-        return trim(substr($text, 0, $limit)) . '...';
+        return trim(substr($text, 0, $limit)).'...';
     }
 }

src/Anroots/Pgca/Rule/Message/NoProfanity.php

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
         $profane = false;
         $message = null;
 
-        $this->profanityChecker->scan($commit->getMessage(), function ($word, $types) use (&$profane, &$message) {
+        $this->profanityChecker->scan($commit->getMessage(), function($word, $types) use (&$profane, &$message) {
             $profane = true;
 
             $message = sprintf(

src/Anroots/Pgca/Rule/Message/AllLinesLessThanThresholdChars.php

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
      */
     public function getMessage()
     {
-        return 'No line should be longer than ' . $this->maxLength . ' characters';
+        return 'No line should be longer than '.$this->maxLength.' characters';
     }
 
     /**