Updated checkstyle library and embrace PER 2.0 (#159)

Author: yann-eugone

src/SkipInvalidItemProcessor.php

@@ -21,11 +21,11 @@ public function __construct(
         /**
          * @var Constraint[]|null
          */
-        private ?array $contraints = null,
+        private null|array $contraints = null,
         /**
          * @var string[]|null
          */
-        private ?array $groups = null,
+        private null|array $groups = null,
     ) {
     }
 
@@ -47,7 +47,7 @@ public function process(mixed $item): mixed
      *
      * @return Iterator<string>
      */
-    private function normalizeConstraints(?array $constraints): Iterator
+    private function normalizeConstraints(null|array $constraints): Iterator
     {
         foreach ($constraints ?? [] as $constraint) {
             yield $constraint::class;

src/SkipItemOnViolations.php

@@ -20,7 +20,7 @@ public function __construct(
         /**
          * @var ConstraintViolationListInterface<ConstraintViolationInterface>
          */
-        private ConstraintViolationListInterface $violations
+        private ConstraintViolationListInterface $violations,
     ) {
     }
 
@@ -34,16 +34,16 @@ public function report(JobExecution $execution, int|string $index, mixed $item):
                 '%s: %s (invalid value: %s)',
                 $violation->getPropertyPath(),
                 $violation->getMessage(),
-                $this->normalizeInvalidValue($violation->getInvalidValue())
+                $this->normalizeInvalidValue($violation->getInvalidValue()),
             );
         }
 
         $execution->addWarning(
             new Warning(
                 'Violations were detected by validator.',
                 [],
-                ['itemIndex' => $index, 'item' => $item, 'violations' => $violations]
-            )
+                ['itemIndex' => $index, 'item' => $item, 'violations' => $violations],
+            ),
         );
     }
 

tests/SkipInvalidItemProcessorTest.php

@@ -19,7 +19,7 @@ class SkipInvalidItemProcessorTest extends TestCase
     /**
      * @dataProvider groups
      */
-    public function testProcessValid(?array $groups): void
+    public function testProcessValid(null|array $groups): void
     {
         $validator = Validation::createValidator();
         $processor = new SkipInvalidItemProcessor($validator, [new NotBlank(['groups' => $groups])], $groups);
@@ -29,12 +29,13 @@ public function testProcessValid(?array $groups): void
     /**
      * @dataProvider groups
      */
-    public function testProcessInvalid(?array $groups): void
+    public function testProcessInvalid(null|array $groups): void
     {
         $validator = Validation::createValidator();
         $processor = new SkipInvalidItemProcessor($validator, [new Blank(['groups' => ['Default', 'Full']])], $groups);
 
         $exception = null;
+
         try {
             $processor->process('invalid item not blank');
         } catch (SkipItemException $exception) {

tests/SkipItemOnViolationsTest.php

@@ -30,7 +30,7 @@ public function test(array $violations, array $expectedViolations): void
         self::assertSame([], $execution->getWarnings()[0]->getParameters());
         self::assertSame(
             ['itemIndex' => 'itemIndex', 'item' => 'item', 'violations' => $expectedViolations],
-            $execution->getWarnings()[0]->getContext()
+            $execution->getWarnings()[0]->getContext(),
         );
     }