Fix doctrine/coding-standard violations (#674)

* Fix doctrine/coding-standard violations * Remove prefer-dist option for composer
doctrine · May 21, 2021 · 904fa3e · 904fa3e
1 parent 8db802c
commit 904fa3e
Show file tree

Hide file tree

Showing 5 changed files with 3 additions and 5 deletions.
diff --git a/.github/workflows/continuous-integration.yml b/.github/workflows/continuous-integration.yml
@@ -84,7 +84,6 @@ jobs:
         uses: "ramsey/composer-install@v1"
         with:
           dependency-versions: "${{ matrix.dependencies }}"
-          composer-options: "--prefer-dist"
 
       - name: "Run PHPUnit with coverage"
         run: "vendor/bin/phpunit --coverage-clover=coverage.xml"

diff --git a/Command/DoctrineODMCommand.php b/Command/DoctrineODMCommand.php
@@ -92,6 +92,7 @@ protected function findBundle($bundleName)
             assert($bundle instanceof Bundle);
             if (strtolower($bundleName) === strtolower($bundle->getName())) {
                 $foundBundle = $bundle;
+
                 break;
             }
         }

diff --git a/Command/TailCursorDoctrineODMCommand.php b/Command/TailCursorDoctrineODMCommand.php
@@ -87,6 +87,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
                 $processor->process($document);
             } catch (Throwable $e) {
                 $output->writeln(sprintf('Error occurred while processing document: <error>%s</error>', $e->getMessage()));
+
                 continue;
             }
 

diff --git a/Loader/SymfonyFixturesLoader.php b/Loader/SymfonyFixturesLoader.php
@@ -109,6 +109,7 @@ public function getFixtures(array $groups = [])
                 $fixtureClass = get_class($fixture);
                 if (isset($this->groupsFixtureMapping[$group][$fixtureClass])) {
                     $filteredFixtures[$fixtureClass] = $fixture;
+
                     continue 2;
                 }
             }

diff --git a/phpcs.xml.dist b/phpcs.xml.dist
@@ -32,10 +32,6 @@
         <exclude name="SlevomatCodingStandard.Classes.SuperfluousTraitNaming" />
     </rule>
 
-    <rule ref="SlevomatCodingStandard.Classes.UnusedPrivateElements.UnusedProperty">
-        <exclude-pattern>Tests/DependencyInjection/Fixtures/*</exclude-pattern>
-    </rule>
-
     <rule ref="PSR1.Classes.ClassDeclaration.MultipleClasses">
         <exclude-pattern>Tests/*</exclude-pattern>
     </rule>