minor #265 Rebuild CI and fix CS (bocharsky-bw)

This PR was squashed before being merged into the main branch.

Discussion
----------

Rebuild CI and fix CS

After some changes upstream in GitHub CI config: SymfonyCasts/.github#2

Commits
-------

16a7ae4 Rebuild CI and fix CS

Author: weaverryan

.php-cs-fixer.dist.php

@@ -19,7 +19,9 @@
 For the full copyright and license information, please view the LICENSE
 file that was distributed with this source code.
 EOF
-        ]
+        ],
+        // Because of the commented out argument in ResetPasswordHelperInterface
+        'no_superfluous_phpdoc_tags' => false,
     ))
     ->setRiskyAllowed(true)
     ->setFinder($finder)

src/Command/ResetPasswordRemoveExpiredCommand.php

@@ -29,17 +29,11 @@ public function __construct(ResetPasswordCleaner $cleaner)
         parent::__construct('reset-password:remove-expired');
     }
 
-    /**
-     * {@inheritdoc}
-     */
     protected function configure(): void
     {
         $this->setDescription('Remove expired reset password requests from persistence.');
     }
 
-    /**
-     * {@inheritdoc}
-     */
     protected function execute(InputInterface $input, OutputInterface $output): int
     {
         $output->writeln('Removing expired reset password requests...');

src/ResetPasswordHelper.php

@@ -60,7 +60,7 @@ public function __construct(ResetPasswordTokenGenerator $generator, ResetPasswor
      *
      * @throws TooManyPasswordRequestsException
      */
-    public function generateResetToken(object $user, ?int $resetRequestLifetime = null): ResetPasswordToken
+    public function generateResetToken(object $user, int $resetRequestLifetime = null): ResetPasswordToken
     {
         $this->resetPasswordCleaner->handleGarbageCollection();
 
@@ -94,8 +94,6 @@ public function generateResetToken(object $user, ?int $resetRequestLifetime = nu
     }
 
     /**
-     * {@inheritdoc}
-     *
      * @throws ExpiredResetPasswordTokenException
      * @throws InvalidResetPasswordTokenException
      */
@@ -133,8 +131,6 @@ public function validateTokenAndFetchUser(string $fullToken): object
     }
 
     /**
-     * {@inheritdoc}
-     *
      * @throws InvalidResetPasswordTokenException
      */
     public function removeResetRequest(string $fullToken): void
@@ -148,9 +144,6 @@ public function removeResetRequest(string $fullToken): void
         $this->repository->removeResetPasswordRequest($request);
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function getTokenLifetime(): int
     {
         return $this->resetRequestLifetime;
@@ -166,7 +159,7 @@ public function getTokenLifetime(): int
      *
      * This method should not be used when timing attacks are a concern.
      */
-    public function generateFakeResetToken(?int $resetRequestLifetime = null): ResetPasswordToken
+    public function generateFakeResetToken(int $resetRequestLifetime = null): ResetPasswordToken
     {
         $resetRequestLifetime = $resetRequestLifetime ?: $this->resetRequestLifetime;
         $expiresAt = new \DateTimeImmutable(sprintf('+%d seconds', $resetRequestLifetime));

tests/FunctionalTests/Persistence/ResetPasswordRequestRepositoryTest.php

@@ -36,9 +36,6 @@ final class ResetPasswordRequestRepositoryTest extends TestCase
      */
     private $repository;
 
-    /**
-     * {@inheritdoc}
-     */
     protected function setUp(): void
     {
         $kernel = new ResetPasswordTestKernel();

tests/UnitTests/Controller/ResetPasswordControllerTraitTest.php

@@ -33,9 +33,6 @@ class ResetPasswordControllerTraitTest extends TestCase
      */
     private $mockSession;
 
-    /**
-     * {@inheritdoc}
-     */
     protected function setUp(): void
     {
         $this->mockSession = $this->createMock(SessionInterface::class);

tests/UnitTests/Generator/ResetPasswordTokenGeneratorTest.php

@@ -30,9 +30,6 @@ class ResetPasswordTokenGeneratorTest extends TestCase
      */
     private $mockExpiresAt;
 
-    /**
-     * {@inheritdoc}
-     */
     protected function setUp(): void
     {
         $this->mockRandomGenerator = $this->createMock(ResetPasswordRandomGenerator::class);

tests/UnitTests/ResetPasswordHelperTest.php

@@ -52,9 +52,6 @@ class ResetPasswordHelperTest extends TestCase
      */
     private $randomToken;
 
-    /**
-     * {@inheritdoc}
-     */
     protected function setUp(): void
     {
         $this->mockRepo = $this->createMock(ResetPasswordRequestRepositoryInterface::class);

tests/UnitTests/Util/ResetPasswordCleanerTest.php

@@ -25,9 +25,6 @@ class ResetPasswordCleanerTest extends TestCase
      */
     private $mockRepo;
 
-    /**
-     * {@inheritdoc}
-     */
     protected function setUp(): void
     {
         $this->mockRepo = $this->createMock(ResetPasswordRequestRepositoryInterface::class);