Cleanup

Author: kubawerlos

README.md

@@ -31,8 +31,8 @@ In your PHP CS Fixer configuration register fixers and use them:
 
 
 ## Fixers
-- **ImplodeCallFixer** - function `implode` must be called with 2 arguments in the documented order. DEPRECATED: use `implode_call` instead.
-
+- **ImplodeCallFixer** - function `implode` must be called with 2 arguments in the documented order.  
+  DEPRECATED: use `implode_call` instead.  
   *Risky: when the function `implode` is overridden.*
 ```diff
  <?php
@@ -83,8 +83,7 @@ In your PHP CS Fixer configuration register fixers and use them:
  $y = new \Baz();
 ```
 
-- **NoNullableBooleanTypeFixer** - there must be no nullable boolean type.
-
+- **NoNullableBooleanTypeFixer** - there must be no nullable boolean type.  
   *Risky: when the null is used.*
 ```diff
  <?php
@@ -107,16 +106,16 @@ In your PHP CS Fixer configuration register fixers and use them:
  namespace Foo;
 ```
 
-- **NoReferenceInFunctionDefinitionFixer** - there must be no reference in function definition.
-
+- **NoReferenceInFunctionDefinitionFixer** - there must be no reference in function definition.  
   *Risky: when rely on reference.*
 ```diff
  <?php
 -function foo(&$x) {}
 +function foo($x) {}
 ```
 
-- **NoTwoConsecutiveEmptyLinesFixer** - there must be no two consecutive empty lines in code. DEPRECATED: use `no_extra_blank_lines` instead.
+- **NoTwoConsecutiveEmptyLinesFixer** - there must be no two consecutive empty lines in code.  
+  DEPRECATED: use `no_extra_blank_lines` instead.
 ```diff
  <?php
  namespace Foo;
@@ -127,8 +126,9 @@ In your PHP CS Fixer configuration register fixers and use them:
 
 - **NoUnneededConcatenationFixer** - there should not be inline concatenation of strings.
 ```diff
--<?php 'foo' . 'bar';
-+<?php 'foobar';
+ <?php
+-echo 'foo' . 'bar';
++echo 'foobar';
 ```
 
 - **NoUselessClassCommentFixer** - there must be no comment like: "Class FooBar".

src/Fixer/NoUnneededConcatenationFixer.php

@@ -15,7 +15,7 @@ public function getDefinition(): FixerDefinition
     {
         return new FixerDefinition(
             'There should not be inline concatenation of strings.',
-            [new CodeSample("<?php 'foo' . 'bar';\n")]
+            [new CodeSample("<?php\necho 'foo' . 'bar';\n")]
         );
     }
 

src/Readme/ReadmeCommand.php

@@ -129,16 +129,14 @@ private function fixers(): string
 
             if ($fixer instanceof DeprecatedFixerInterface) {
                 $output .= \sprintf(
-                    ' DEPRECATED: use `%s` instead.',
+                    "  \n  DEPRECATED: use `%s` instead.",
                     \implode('`, `', $fixer->getSuccessorsNames())
                 );
             }
 
-            $output .= "\n";
-
             if ($fixer->isRisky()) {
                 $output .= \sprintf(
-                    "\n  *Risky: %s*\n",
+                    "  \n  *Risky: %s*",
                     \lcfirst($fixer->getDefinition()->getRiskyDescription())
                 );
             }
@@ -150,7 +148,7 @@ private function fixers(): string
             $fixedCode = $tokens->generateCode();
 
             $output .= \sprintf(
-                "```diff\n%s\n```\n",
+                "\n```diff\n%s\n```\n",
                 $this->diff($originalCode, $fixedCode)
             );
         }

tests/Fixer/NoImportFromGlobalNamespaceFixerTest.php

@@ -147,7 +147,7 @@ public function __construct() {
             '<?php
 namespace Foo;
 class Bar {
-    /** 
+    /**
      * @param \DateTime $a
      * @param \DateTime $b
      * @param NotDateTime $c
@@ -165,7 +165,7 @@ public function __construct($a, $b, $c, $d, $e, $f, $g, $h, $i) {}
 namespace Foo;
 use DateTime;
 class Bar {
-    /** 
+    /**
      * @param DateTime $a
      * @param \DateTime $b
      * @param NotDateTime $c

tests/Fixer/NoUnneededConcatenationFixerTest.php

@@ -41,7 +41,7 @@ public function provideFixCases(): \Iterator
         yield ['<?php "foo".\'bar\';'];
         yield ['<?php "foo" . \'bar\';'];
         yield ['<?php \'bar\' . "foo";'];
-        yield ['<?php "foo" 
+        yield ['<?php "foo"
                       . "bar";'];
         yield ['<?php "foo" .
                       "bar";'];

tests/Fixer/PhpdocSelfAccessorFixerTest.php

@@ -41,7 +41,7 @@ class Foo {
      * @var FooBar
      */
      private $instance;
-     
+
      /**
       * @param Foo\Bar $x
       * @param Bar\Foo $x
@@ -59,7 +59,7 @@ class Foo {
      * @var self
      */
      private $instance;
-     
+
      /**
       * @param self $x
       * @param self $x
@@ -82,7 +82,7 @@ class Foo {
      * @var Foo
      */
      private $instance;
-     
+
      /**
       * @param Foo $x
       * @param \Foo $x