Fix the errors/warnings reported by PHP_CodeSniffer

The provided patch file "fix_coding_standards-3394540-4.patch", fixes the --standard=Drupal,DrupalPractice issues reported by phpcs.

- * Class ResendController.
+ * Implements ResendController Class.

Adding Implements does not change the fact the description is merely repeating the class name. That comment also became not correct, as a class does not implement itself.

+  /**
+   * Implements MessengerInterface.
+   *
+   * @var Drupal\Core\Messenger\MessengerInterface
+   */
   protected $messenger;

A property description must describe what the property is, not which interfaces it implements, which is information already given in the @var line.

+  /**
+   * Construct a new ResendController object.
+   */
   public function __construct(MessengerInterface $messenger) {
     $this->messenger = $messenger;
   }

The documentation comment for constructors is not mandatory anymore, If it is given, the description must be Constructs a new [class name] object. where [class name] includes the class namespace. The parameter descriptions must be included too.

-        $this->logger->notice('Password reset instructions mailed to %name at %email.', ['%name' => $account->getAccountName(), '%email' => $account->getEmail()]);
+        $this->logger->notice('Password reset instructions mailed to %name at %email.',
+         [
+           '%name' => $account->getAccountName(),
+           '%email' => $account->getEmail(),
+         ]);

Code lines can exceed the 80 characters, if they are easier to understand. The existing code is easier to understand.

+   * @param \Drupal\user\UserStorageInterface $user_storage
+   *   User storage.

The description is missing a definite article.

Revised the Patch "fix_coding_standards-3394540-4.patch" as per Comment #5. The patch file "fix_coding_standards-3394540-6.patch" fixes the issues reported by phpcs in accordance with Comment #5.

+  /**
+   * Provides access to the Messenger service for displaying messages.
+   *
+   * @var Drupal\Core\Messenger\MessengerInterface
+   */
   protected $messenger;

The messenger. is sufficient as description.

+   * @param \Drupal\Core\Messenger\MessengerInterface $messenger
+   *   The messenger service.

It is not necessary to say it is a service.

+  /**
+   * Implements resend().
+   */

That description is merely repeating the method name.
The parameter descriptions and the return value description are missing.

+   * @param \Drupal\user\UserStorageInterface $user_storage
+   *   A User storage.

User does not need to be capitalized. The descriptions start with a definite article.

Since there is a 9.x branch, that is probably the branch that should be changed. At least, that should be the first branch to be changed and, eventually, the changes back-ported to the other branch.

MR Updated.
Please review.

MR Updated for review.

Thanks for all the work here @avpaderno.