Fix the issues reported by phpcs

Hi Jay Jangid

I applied your MR !2 in my local and it applied cleanly, however, there are still PHPCS errors being flagged,

contrib git:(master) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml monix

FILE: /Users/studenttrainees/Projects/drupalorgissues/docroot/modules/contrib/monix/src/MonixPluginManager.php
--------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------------------------------------
48 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
--------------------------------------------------------------------------------------------------------------

FILE: /Users/studenttrainees/Projects/drupalorgissues/docroot/modules/contrib/monix/src/Controller/MonixController.php
----------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 3 LINES
----------------------------------------------------------------------------------------------------------------------
77 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
99 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
100 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
----------------------------------------------------------------------------------------------------------------------

Time: 210ms; Memory: 10MB

Please look at the screenshot attached for your reference.

Thanks

I will work on this issue.

created a patch to fix the issue. one is remaining. please reivew

I do not have time for a full review, but the following property is already defined in the parent class (ControllerBase) and it does not need to be redefined from this class.

+  /**
+   * The configuration factory service.
+   *
+   * @var \Drupal\Core\Config\ConfigFactoryInterface
+   */
+  protected $configFactory;

I would suggest to check the changes done to avoid redefining properties the parent classes already define and verify which methods from the parent classes can be used.

Patch updated.
Please review.

+  /**
+   * The current global container.
+   *
+   * @var \Drupal\Component\DependencyInjection\Container
+   */
+  protected $container;
+

The dependency injection container. is a better description, since current global container does not make clear which container that would be.

Patch updated as per suggestions.
Please review.

Hi, Reviewed the patch at #13 applies cleanly, fixes all the phpcs errors and addresses the comment at #11.
Thank you!