Fix the issues reported by phpcs

Rerolled as MR and fixed everything except

FILE: /Users/tesboss/_PROJECTS/drupal/viewfield/src/Plugin/Field/FieldFormatter/ViewfieldFormatterDefault.php
-------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------------------
 249 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
-------------------------------------------------------------------------------------------------------------

MR 6 review and tested locally and PHPCS issues are resolved. Please check the screenshot for reference.

The issue summary should always describe what the issue is trying to fix and, in the case, of coding standards issues, show which command has been used, which arguments have been used, and which report that command shown.

+  /**
+   * The Token service.
+   *
+   * @var \Drupal\Core\Utility\Token

Only the first word in the description must be capitalized.

+  /**
+   * Constructs a StringFormatter instance.

The class is not correct; the class is ViewfieldFormatterDefault.
The description for a constructor must start with Constructs a new followed by the class name (including its namespace), and end with object.

-        // Set arguments if they exist
+        // Set arguments if they exist.
         if (!empty($arguments)) {
           $view->setArguments($arguments);
         }

That comment can be removed, since it describes something already clear from the code.

+ // @todo Change the autogenerated stub

The period at the end is missing.

 /**
- * Service Provider for Viewfield
+ * Service Provider for Viewfield.
  */
 class ViewfieldServiceProvider extends ServiceProviderBase {

That description does not say what the class does.
Only the first word must be capitalized.

Updated the MR with the changes mentioned in #26. Please review.

-        // Set arguments if they exist
+        // Set arguments if they exist.
         if (!empty($arguments)) {
           $view->setArguments($arguments);
         }

That comment can be removed, since it describes something already clear from the code.

- * Service Provider for Viewfield
+ * Sets definition for searializer and tag for normalizer.

There is a typo in that description, which does not make sense.

+  /**
+   * Test view field item default value.
+   */

The verb must use the third singular person.
The description should be rewritten: Four attributive nouns are too much.

-  //  if (!isset($default_attributes)) {
-  //    $default_attributes = new Attribute();
-  //  }
+  // if (!isset($default_attributes)) {
+  // $default_attributes = new Attribute();
+  // }

Even if commented out, that is still code. The existing line are correctly indented; the changed code is not correctly indented.

-    //    $variables['items'][$delta]['attributes'] = !empty($element['#items'][$delta]->_attributes) ? new Attribute($element['#items'][$delta]->_attributes) : clone($default_attributes);
+    // $variables['items'][$delta]['attributes'] =
+    // !empty($element['#items'][$delta]->_attributes) ?
+    // new Attribute($element['#items'][$delta]->_attributes) :
+    // clone($default_attributes);

Code lines can exceed 80 characters, if they are more readable. That is still valid for commented out code.