Fix the issues reported by phpcs

I have Created a patch to fix this issue.

Why is --standard=Drupal used instead of --standard=Drupal,DrupalPractice?

Hii @apaderno,
I addressed DrupalPractice issue here https://www.drupal.org/project/fakelogin/issues/3362606 📌 \Drupal::service('router.builder')->rebuild() call should be avoided in classes, Use dependency injection instead in drupal Database Closed: duplicate

-    // replace default user login with a fake form.
+    // Replace default user login with a fake form.

Since that comment is edited, a definite article should be added before default user; form should be added after user login.

Tried to fixes the issue.
providing patch please review.

Hello Folks !!
I have tested the #6 patch and the phpcs issues has been successfully fixed! The patch was tested on Drupal 9.5.9 and PHP 8.2.4. To provide you with a comprehensive view, I will be adding both a before and after screenshot.
Thank you for your support!

This is the screenshot before applying the patch!!

+  /**
+   * {@inheritdoc}
+   */
+  public function __construct(RouteBuilderInterface $routeBuilder) {

The description for a constructor must start with Constructs a new followed by the class name (including its namespace), and end with object.
The documentation comment for a constructor must describe the parameters.

-    //$this->messenger()->addStatus($this->t('Your username is @username', ['@username' => $form_state->getValue('name')]));
+    // $this->messenger()->addStatus($this->t('Your username is @username', [
+    // '@username' => $form_state->getValue('name')
+    // ]));

Commented out code must still be correctly indented.

-	  //Route options
+      // Route options.
       [
         '_maintenance_access' => TRUE,
-      ]	  
+      ]

Rather than editing the comment, I would remove it.

-    // replace default user login with a fake form.
+    // Replace default user login with a fake form.

The definite article is missing before default user login. form is missing after default user login.

Addressed #9.
Updated patch.

+  /**
+   * Constructs a new FakeLoginConfigForm instance.

The class namespace is missing.

-    // rebuild the router.
-    \Drupal::service('router.builder')->rebuild();
+    // Rebuild the router.
+    $this->routeBuilder->rebuild();
 

I would rather remove that comment, as $this->routeBuilder->rebuild(); does not need explanations.

-    //$this->messenger()->addStatus($this->t('Your username is @username', ['@username' => $form_state->getValue('name')]));
+    // $this->messenger()->addStatus($this->t('Your username is @username', [
+    // '@username' => $form_state->getValue('name')
+    // ]));

Commented out code still needs to be indented how code should be intended. (That means that '@username' => $form_state->getValue('name') must be indented by two spaces more.)
I would rather remove that commented-out code as it seems more debugging code that should not be left in a public release.

-    // replace default user login with a fake form.
+    // Replace the default user login with a fake form.

form is still missing after default user login.

Updated patch #11 by addressing #13, please review it.

Thanks!

fixed.

Automatically closed - issue fixed for 2 weeks with no activity.