Fix CS

overblog · Jan 29, 2024 · ca5ec9f · ca5ec9f
1 parent ddb6c93
commit ca5ec9f
Show file tree

Hide file tree

Showing 9 changed files with 6 additions and 36 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -116,7 +116,7 @@ jobs:
       - name: "Install PHP"
         uses: "shivammathur/setup-php@v2"
         with:
-          php-version: "8.2"
+          php-version: "8.1"
           coverage: "none"
 
       - name: "Install dependencies"

diff --git a/src/Config/TypeDefinition.php b/src/Config/TypeDefinition.php
@@ -81,6 +81,7 @@ protected function validationSection(int $level): ArrayNodeDefinition
                                 return $value;
                             }
                         }
+
                         // validation: [list of constraints]
                         return ['constraints' => $value];
                     }

diff --git a/src/Relay/Connection/ConnectionInterface.php b/src/Relay/Connection/ConnectionInterface.php
@@ -27,8 +27,6 @@ public function setEdges(iterable $edges): void;
 
     /**
      * Get the page info.
-     *
-     * @return PageInfoInterface
      */
     public function getPageInfo(): ?PageInfoInterface;
 

diff --git a/src/Relay/Connection/Output/PageInfo.php b/src/Relay/Connection/Output/PageInfo.php
@@ -23,9 +23,6 @@ public function __construct(string $startCursor = null, string $endCursor = null
         $this->hasNextPage = $hasNextPage;
     }
 
-    /**
-     * @return string
-     */
     public function getStartCursor(): ?string
     {
         return $this->startCursor;
@@ -36,9 +33,6 @@ public function setStartCursor(string $startCursor): void
         $this->startCursor = $startCursor;
     }
 
-    /**
-     * @return string
-     */
     public function getEndCursor(): ?string
     {
         return $this->endCursor;
@@ -49,9 +43,6 @@ public function setEndCursor(string $endCursor): void
         $this->endCursor = $endCursor;
     }
 
-    /**
-     * @return bool
-     */
     public function getHasPreviousPage(): ?bool
     {
         return $this->hasPreviousPage;
@@ -62,9 +53,6 @@ public function setHasPreviousPage(bool $hasPreviousPage): void
         $this->hasPreviousPage = $hasPreviousPage;
     }
 
-    /**
-     * @return bool
-     */
     public function getHasNextPage(): ?bool
     {
         return $this->hasNextPage;

diff --git a/src/Relay/Connection/PageInfoInterface.php b/src/Relay/Connection/PageInfoInterface.php
@@ -12,30 +12,18 @@
 
 interface PageInfoInterface
 {
-    /**
-     * @return string
-     */
     public function getStartCursor(): ?string;
 
     public function setStartCursor(string $startCursor): void;
 
-    /**
-     * @return string
-     */
     public function getEndCursor(): ?string;
 
     public function setEndCursor(string $endCursor): void;
 
-    /**
-     * @return bool
-     */
     public function getHasPreviousPage(): ?bool;
 
     public function setHasPreviousPage(bool $hasPreviousPage): void;
 
-    /**
-     * @return bool
-     */
     public function getHasNextPage(): ?bool;
 
     public function setHasNextPage(bool $hasNextPage): void;

diff --git a/src/Resolver/TypeResolver.php b/src/Resolver/TypeResolver.php
@@ -46,8 +46,6 @@ protected function onLoadSolution($solution): void
 
     /**
      * @param string $alias
-     *
-     * @return Type
      */
     public function resolve($alias): ?Type
     {

diff --git a/src/Upload/Type/GraphQLUploadType.php b/src/Upload/Type/GraphQLUploadType.php
@@ -14,9 +14,6 @@
 
 final class GraphQLUploadType extends ScalarType
 {
-    /**
-     * @param string $name
-     */
     public function __construct(string $name = null)
     {
         parent::__construct([

diff --git a/src/Validator/InputValidator.php b/src/Validator/InputValidator.php
@@ -63,7 +63,7 @@ public function __construct(
     /**
      * @throws ArgumentsValidationException
      */
-    public function validate(string|array|null $groups = null, bool $throw = true): ?ConstraintViolationListInterface
+    public function validate(string|array $groups = null, bool $throw = true): ?ConstraintViolationListInterface
     {
         $rootNode = new ValidationNode(
             $this->info->parentType,

diff --git a/tests/Config/Parser/fixtures/graphql/schema.php b/tests/Config/Parser/fixtures/graphql/schema.php
@@ -159,9 +159,9 @@
         'type' => 'custom-scalar',
         'config' => [
             'description' => null,
-            'serialize' => [\Overblog\GraphQLBundle\Config\Parser\GraphQL\ASTConverter\CustomScalarNode::class, 'mustOverrideConfig'],
-            'parseValue' => [\Overblog\GraphQLBundle\Config\Parser\GraphQL\ASTConverter\CustomScalarNode::class, 'mustOverrideConfig'],
-            'parseLiteral' => [\Overblog\GraphQLBundle\Config\Parser\GraphQL\ASTConverter\CustomScalarNode::class, 'mustOverrideConfig'],
+            'serialize' => [Overblog\GraphQLBundle\Config\Parser\GraphQL\ASTConverter\CustomScalarNode::class, 'mustOverrideConfig'],
+            'parseValue' => [Overblog\GraphQLBundle\Config\Parser\GraphQL\ASTConverter\CustomScalarNode::class, 'mustOverrideConfig'],
+            'parseLiteral' => [Overblog\GraphQLBundle\Config\Parser\GraphQL\ASTConverter\CustomScalarNode::class, 'mustOverrideConfig'],
         ],
     ],
 ];
-Original file line number
+Diff line change
@@ Expand Up @@
                                     return $value;
                                 }
                             }
                             // validation: [list of constraints]
                             return ['constraints' => $value];
                         }
@@ Expand Down @@