Realodix: Remove concat_space rule, aligning with Laravel defaults

This commit removes the explicit `concat_space` rule, effectively setting it to 'none' and aligning with Laravel's coding standards.

Author: realodix

.php-cs-fixer.dist.php

@@ -16,4 +16,4 @@
     ->setRules($localRules)
     ->setFinder($finder)
     ->setParallelConfig(\PhpCsFixer\Runner\Parallel\ParallelConfigFactory::detect())
-    ->setCacheFile(__DIR__ . '/.tmp/.php-cs-fixer.cache');
+    ->setCacheFile(__DIR__.'/.tmp/.php-cs-fixer.cache');

bin/relax

@@ -6,7 +6,7 @@
     $require = true;
     if ($require) {
         // OK, it's not, let give Composer autoloader a try!
-        $possibleFiles = [__DIR__ . '/../../../autoload.php', __DIR__ . '/../vendor/autoload.php', __DIR__ . '/vendor/autoload.php'];
+        $possibleFiles = [__DIR__.'/../../../autoload.php', __DIR__.'/../vendor/autoload.php', __DIR__.'/vendor/autoload.php'];
         $file = null;
         foreach ($possibleFiles as $possibleFile) {
             if (file_exists($possibleFile)) {

src/Commands/GenerateConfigCommand.php

@@ -65,7 +65,7 @@ protected function overwriteExistingFile($input, $output): bool
      */
     protected function getOutputFilename(): string
     {
-        return getcwd() . DIRECTORY_SEPARATOR . self::FILE_NAME;
+        return getcwd().DIRECTORY_SEPARATOR.self::FILE_NAME;
     }
 
     /**

src/Config.php

@@ -53,7 +53,7 @@ public static function create($ruleSet = null)
 
         // If the rule set is a string, we try to find it in the RuleSet namespace
         if (is_string($ruleSet)) {
-            $relaxRuleset = 'Realodix\Relax\RuleSet\Sets\\' . ucfirst($ruleSet);
+            $relaxRuleset = 'Realodix\Relax\RuleSet\Sets\\'.ucfirst($ruleSet);
 
             if (!class_exists($relaxRuleset)) {
                 throw new RulesetNotFoundException($ruleSet);

src/RuleSet/AbstractRuleSet.php

@@ -10,6 +10,6 @@ public function name(): string
     {
         $name = substr(static::class, 1 + strrpos(static::class, '\\'));
 
-        return '@' . $name;
+        return '@'.$name;
     }
 }

src/RuleSet/Sets/Relax.php

@@ -62,7 +62,6 @@ public function mainRules(): array
                 'default' => 'single_space',
                 'operators' => ['=>' => 'at_least_single_space'],
             ],
-            'concat_space' => ['spacing' => 'one'],
             'class_attributes_separation' => [
                 'elements' => [
                     'trait_import' => 'none',

tests/Fixtures/Ruleset/relax-commonbox_expected.php

@@ -217,7 +217,7 @@ final class SelfStaticAccessor
 
     public function getBar()
     {
-        return self::class . self::test() . self::$A;
+        return self::class.self::test().self::$A;
     }
 }
 

tests/Fixtures/Ruleset/relax_actual.php

@@ -347,7 +347,7 @@ public function operator__binary_operator_spaces()
 
     public function operator__concat_space()
     {
-        $foo = 'bar'. 3 . 'baz'.'qux';
+        return 'bar'. 3 . 'baz'.'qux';
     }
 
     public function operator__increment_style()

tests/Fixtures/Ruleset/relax_expected.php

@@ -353,7 +353,7 @@ public function operator__binary_operator_spaces()
 
     public function operator__concat_space()
     {
-        $foo = 'bar' . 3 . 'bazqux';
+        return 'bar'. 3 .'bazqux';
     }
 
     public function operator__increment_style()

tests/Integration/IntegrationTestCase.php

@@ -25,19 +25,19 @@ protected function testFixture(string $name, ?string $suffix = null): void
         $fileName = $name;
 
         if (!is_null($suffix)) {
-            $fileName = $fileName . '-' . $suffix;
+            $fileName = $fileName.'-'.$suffix;
         }
 
-        copy(__DIR__ . "/../Fixtures/Ruleset/{$fileName}_actual.php", __DIR__ . "/tmp/{$fileName}.php");
+        copy(__DIR__."/../Fixtures/Ruleset/{$fileName}_actual.php", __DIR__."/tmp/{$fileName}.php");
 
         $this->assertTrue(
             $this->runFixer($name),
             "Fixture fixtures/{$fileName} was not proceeded properly.",
         );
 
         $this->assertFileEquals(
-            __DIR__ . "/../Fixtures/Ruleset/{$fileName}_expected.php",
-            __DIR__ . "/tmp/{$fileName}.php",
+            __DIR__."/../Fixtures/Ruleset/{$fileName}_expected.php",
+            __DIR__."/tmp/{$fileName}.php",
             "Result of proceeded fixture fixtures/{$fileName} is not equal to expected.",
         );
     }
@@ -61,7 +61,7 @@ protected function runFixer(string $name): bool
 
     protected function clearTempDirectory(): void
     {
-        $files = glob(__DIR__ . '/tmp/*.php');
+        $files = glob(__DIR__.'/tmp/*.php');
 
         foreach ($files as $file) {
             unlink($file);