Fix the issues reported by phpcs

Above errors/warnings has been fixed.

Hi samit.310@gmail.com

I applied patch #2 to the “Keyboard shortcuts” module against Version 1.1.x-dev and confirmed that the errors and warnings are resolved. Please see the screenshots attached.

For your review.
Thank you.

+/**
+ * Admin config form for Keyboard shortcuts module.
+ */
 class KeyboardShortcutsSettingsForm extends ConfigFormBase {/

Description for classes must not say which module implements them.

+  /**
+   * Constructs a KeyboardShortcutsSettingsForm object.
+   *
+   * @param \Drupal\Core\Config\ConfigFactoryInterface $config_factory
+   *   The factory for configuration objects.
+   * @param \Drupal\Core\Entity\EntityTypeManagerInterface $entityTypeManager
+   *   The entityTypeManager.
+   */
+  public function __construct(ConfigFactoryInterface $config_factory,
+                              EntityTypeManagerInterface $entityTypeManager) {

The description for a constructor must start with Constructs a new followed by the class name (including its namespace), and end with object.
Method declarations go on a single line.

    * @return \Drupal\Core\StringTranslation\TranslatableMarkup
+   *   Description text.
    */
   protected function getDescription() {

I am not sure that description is sufficient, but it misses a definite article.

+  /**
+   * Add callback.
+   */
   public function addCallback(array &$form, FormStateInterface $form_state) {

The parameter descriptions are missing.

+  /**
+   * Ajex Callback.
+   */

There is a typo and a work that must be spelled in lowercase letters.

+  /**
+   * Get the new shortcut.
+   */
   protected function getNewShortcut() {

The return value is not described.

Patch updated and interdiff attached.
Only one warning showing up because of the contructor description including namespace of the class, assuming this can be ignored.

Please review.

❯ phpcs --standard="Drupal,DrupalPractice" --extensions=php,module,inc,install,test,profile,theme,info,txt,md,yml,css,js .


FILE: /Users/specbee/Sites/Projects/keyboard_shortcuts/src/Form/KeyboardShortcutsSettingsForm.php
-------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------
 24 | WARNING | Line exceeds 80 characters; contains 91 characters
-------------------------------------------------------------------------------------------------

Time: 197ms; Memory: 12MB