Update coding standard (#7)

Author: dantleech

.github/workflows/ci.yml

@@ -43,7 +43,7 @@ jobs:
 
             -
                 name: "Run friendsofphp/php-cs-fixer"
-                run: "vendor/bin/php-cs-fixer fix --dry-run --diff --verbose"
+                run: "vendor/bin/php-cs-fixer fix --dry-run --diff --verbose --allow-risky=yes"
     phpstan:
         name: "PHPStan (${{ matrix.php-version }})"
 

.php_cs.dist

@@ -3,13 +3,22 @@
 $finder = PhpCsFixer\Finder::create()
     ->in('lib')
     ->in('tests')
+    ->exclude([
+        'Workspace',
+    ])
 ;
 
 return PhpCsFixer\Config::create()
     ->setRules([
         '@PSR2' => true,
         'no_unused_imports' => true,
         'array_syntax' => ['syntax' => 'short'],
+        'void_return' => true,
+        'ordered_class_elements' => true,
+        'single_quote' => true,
+        'heredoc_indentation' => true,
+        'global_namespace_import' => true,
     ])
     ->setFinder($finder)
 ;
+

lib/DebugExtension.php

@@ -16,7 +16,7 @@ class DebugExtension implements Extension
     /**
      * {@inheritDoc}
      */
-    public function load(ContainerBuilder $container)
+    public function load(ContainerBuilder $container): void
     {
         $container->register(ExtensionDocumentor::class, function (Container $container) {
             return new ExtensionDocumentor(
@@ -37,7 +37,7 @@ public function load(ContainerBuilder $container)
     /**
      * {@inheritDoc}
      */
-    public function configure(Resolver $schema)
+    public function configure(Resolver $schema): void
     {
     }
 }

lib/bootstrap.php

@@ -13,6 +13,6 @@
     'source' => new SourceContextProvider(),
 ]);
 
-VarDumper::setHandler(function ($var) use ($cloner, $dumper) {
+VarDumper::setHandler(function ($var) use ($cloner, $dumper): void {
     $dumper->dump($cloner->cloneVar($var));
 });