Fix the issues reported by phpcs

Created MR , please review.

Thank you.

Hi Jay Jangid

I applied your MR to the “AMP Optimizer” module and confirmed that the phpcs issues are resolved. Please see the screenshots attached.

For your review.
Thank you.

/**
- * Class OptimizerSubscriber.
- *
- * @package Drupal\amp_optimizer\EventSubscriber
+ * Summary of OptimizerSubscriber.
  */

   /**
-   * @var \Drupal\Core\Session\AccountProxyInterface
+   * Summary of currentUser.
+   *
+   * @var mixed
    */
   protected $currentUser;

+  /**
+   * Summary of __construct.
+   *
+   * @param \AmpProject\Optimizer\TransformationEngine $transformation_engine
+   *   The transformation_engine.
+   * @param \Drupal\Core\Config\ConfigFactoryInterface $config_factory
+   *   The config_factory.
+   * @param \Drupal\Core\Logger\LoggerChannelFactoryInterface $logger_channel_factory
+   *   The logger_channel_factory.
+   * @param \Drupal\Core\Session\AccountProxyInterface $current_user
+   *   The current_user.
+   */
   public function __construct(TransformationEngine $transformation_engine, ConfigFactoryInterface $config_factory, LoggerChannelFactoryInterface $logger_channel_factory, AccountProxyInterface $current_user) {

Summary of is not a description for a class, a method, or a parameter.

Working on this.

Patch added.
Please review.

 /**
- * Class OptimizerSubscriber.
+ * An eventsubscriber of amp_optimizer.
  *

eventsubscriber are two words.
A class description must not include the module machine name; eventually, it should describe the class without using a generic description like An event subscriber.

-   * @param Response $response
-   * @param Request $request
+   * Checks if the response format is html.

As acronym, it is spelled HTML.

    * @return bool
+   *   FALSE if the response is not instance of htmlresponse.
    */

an is missing before instance of. If htmlresponse is the class name, it must be written using the same uppercase/lowercase characters.

Created MR7. Please review.

voleger → closed merge request !6

Oh, closing the issue. Thanks

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