Fix the issues reported by phpcs

Verified the MR. All the issues are resolved, post applying the patch. Attaching screenshot for reference.
Moving to RTBC.

+   * @param string $category_key
+   *   Category key.

That description is not sufficient. It is only repeating the parameter name with two characters less.

-      // @TODO!
-      // $icon_url = \Drupal::service('file_url_generator')->generateString($icon_base_path . 'linuxfoundation.svg');
+      // @todo !
+      // $icon_url = \Drupal::service('file_url_generator')->
+      // generateString($icon_base_path . 'linuxfoundation.svg');

What follows @todo must be a sentence. ! is not a sentence.

+/**
+ * {@inheritDoc}
+ */
 trait MdropSuiteBundleFieldCtaTrait {

{@inheritDoc} is not used in the documentation comment for a trait.

+  /**
+   * The entity definitions update manager service.
+   *
+   * @var \Drupal\Core\Entity\EntityDefinitionUpdateManagerInterface
+   */
   protected $entityDefinitionsUpdateManager;

That is not the correct description.

+  /**
+   * {@inheritDoc}
+   */
   public function installBlockContentBundleFields($bundle_key, $class) {
     $this->installEntityTypeBundleFields('block_content', $bundle_key, $class);
   }

In a class that does not extend another class nor does implement an interface, {@inheritDoc} cannot be used.

 /**
- * Class MdropSuiteLayoutBase.
+ * Class os MdropSuiteLayoutBase.
  */

No, adding a random word between Class and the class name is not sufficient for the comment to just repeat the class name. It is still repeating the class name and not giving any information about the class purpose.

+  /**
+   * {@inheritDoc}
+   */
   abstract protected function getWidthOptions();

I take that is not a method defined in a parent class or an interface. That documentation comment is wrong. {@inheritDoc} is not a replacement for every documentation comment somebody is not able to write.

    * @param string $bundle_key
-   *  Media bundle key.
+   *   Media bundle key.

The description is missing an article. I know that is what the existing comment says, but since the comment is changed, it should be changed to make it correct.

+  /**
+   * Mdrop_suite_modal constructor.
+   *
+   * @param \Drupal\Core\Asset\LibraryDiscoveryInterface $library_discovery
+   *   The LibraryDiscovery service.
+   */
+  public function __construct(LibraryDiscoveryInterface $library_discovery) {

The short description must include the class name, its namespace, and start with a verb, for example Constructs a new \Drupal\user\Entity\User object.

    * @return bool
-   *  Result.
+   *   Result.

That is not a sufficient description for the return value.

- * Class MdropSuiteModalLibraryDiscoveryCollector.
+ * Class LibraryDiscoveryCollector MdropSuiteModalLibraryDiscoveryCollector.

The documentation comment needs to say what the class purpose is, not the class name and the parent class name.

Thanks for your time, unfortunately this module is not longer developed because it has been superseded by VLSuite → .