lint

Author: saeedvaziry

.github/workflows/code-style.yml

@@ -0,0 +1,41 @@
+name: code-style
+
+on:
+  push:
+    branches:
+      - main
+  pull_request:
+
+jobs:
+  code-style:
+    runs-on: ubuntu-22.04
+
+    strategy:
+      fail-fast: true
+      matrix:
+        php: [ 8.2 ]
+        node-version: [ "20.x" ]
+
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: Setup PHP
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: ${{ matrix.php }}
+
+      - name: Cache Composer packages
+        id: composer-cache
+        uses: actions/cache@v2
+        with:
+          path: vendor
+          key: ${{ runner.os }}-php-${{ hashFiles('**/composer.lock') }}
+          restore-keys: |
+            ${{ runner.os }}-php-
+
+      - name: Install dependencies
+        if: steps.composer-cache.outputs.cache-hit != 'true'
+        run: composer install --prefer-dist --no-progress --no-suggest
+
+      - name: Run pint
+        run: ./vendor/bin/pint --test

composer.json

@@ -18,5 +18,8 @@
         "psy/psysh": "*"
     },
     "minimum-stability": "stable",
-    "prefer-stable": true
+    "prefer-stable": true,
+    "require-dev": {
+        "laravel/pint": "^1.19"
+    }
 }

composer.lock

@@ -2,12 +2,12 @@
 
 use TweakPHP\Client\Loader;
 
-require __DIR__ . '/vendor/autoload.php';
+require __DIR__.'/vendor/autoload.php';
 
 $arguments = $argv;
 
 if (count($arguments) < 3) {
-    echo 'Invalid arguments' . PHP_EOL;
+    echo 'Invalid arguments'.PHP_EOL;
     exit(1);
 }
 
@@ -23,7 +23,7 @@ function dd(...$args)
 $loader = Loader::load($arguments[1]);
 
 if ($loader === null) {
-    echo 'Invalid path' . PHP_EOL;
+    echo 'Invalid path'.PHP_EOL;
     exit(1);
 }
 
@@ -34,8 +34,8 @@ function dd(...$args)
     'execute',
 ];
 
-if (!in_array($arguments[2], $supportedCommands)) {
-    echo 'Invalid command' . PHP_EOL;
+if (! in_array($arguments[2], $supportedCommands)) {
+    echo 'Invalid command'.PHP_EOL;
     exit(1);
 }
 
@@ -46,13 +46,13 @@ function dd(...$args)
             'version' => $loader->version(),
             'php_version' => phpversion(),
         ]);
-        echo $info . PHP_EOL;
+        echo $info.PHP_EOL;
         break;
     case 'execute':
         if (count($arguments) < 4) {
-            echo 'Invalid arguments' . PHP_EOL;
+            echo 'Invalid arguments'.PHP_EOL;
             exit(1);
         }
-        echo $loader->execute(base64_decode($arguments[3])) . PHP_EOL;
+        echo $loader->execute(base64_decode($arguments[3])).PHP_EOL;
         break;
-}
+}

index.php

@@ -12,7 +12,6 @@
 class Loader
 {
     /**
-     * @param string $path
      * @return null|LoaderInterface
      */
     public static function load(string $path)

src/Loader.php

@@ -43,7 +43,7 @@ public function init()
             ]);
         }
 
-        $this->tinker = new Tinker(new CustomOutputModifier(), $config);
+        $this->tinker = new Tinker(new CustomOutputModifier, $config);
     }
 
     public function execute(string $code)
@@ -52,4 +52,4 @@ public function execute(string $code)
 
         echo trim($output);
     }
-}
+}

src/Loaders/BaseLoader.php

@@ -4,33 +4,23 @@
 
 class ComposerLoader extends BaseLoader
 {
-    /**
-     * @param string $path
-     * @return bool
-     */
     public static function supports(string $path): bool
     {
-        return file_exists($path . '/vendor/autoload.php');
+        return file_exists($path.'/vendor/autoload.php');
     }
 
-    /**
-     * @param string $path
-     */
     public function __construct(string $path)
     {
-        require $path . '/vendor/autoload.php';
+        require $path.'/vendor/autoload.php';
     }
 
     public function name(): string
     {
         return 'Composer Project';
     }
 
-    /**
-     * @return string
-     */
     public function version(): string
     {
-        return "";
+        return '';
     }
 }

src/Loaders/ComposerLoader.php

@@ -2,42 +2,27 @@
 
 namespace TweakPHP\Client\Loaders;
 
-use Throwable;
-
 class LaravelLoader extends ComposerLoader
 {
     private $app;
 
-    /**
-     * @param string $path
-     * @return bool
-     */
     public static function supports(string $path): bool
     {
-        return file_exists($path . '/vendor/autoload.php') && file_exists($path . '/bootstrap/app.php');
+        return file_exists($path.'/vendor/autoload.php') && file_exists($path.'/bootstrap/app.php');
     }
 
-    /**
-     * @param string $path
-     */
     public function __construct(string $path)
     {
         parent::__construct($path);
-        $this->app = require_once $path . '/bootstrap/app.php';
+        $this->app = require_once $path.'/bootstrap/app.php';
         $this->app->make('Illuminate\Contracts\Console\Kernel')->bootstrap();
     }
 
-    /**
-     * @return string
-     */
     public function name(): string
     {
         return 'Laravel';
     }
 
-    /**
-     * @return string
-     */
     public function version(): string
     {
         return $this->app->version();

src/Loaders/LaravelLoader.php

@@ -4,20 +4,10 @@
 
 interface LoaderInterface
 {
-    /**
-     * @param string $path
-     * @return bool
-     */
     public static function supports(string $path): bool;
 
-    /**
-     * @return string
-     */
     public function name(): string;
 
-    /**
-     * @return string
-     */
     public function version(): string;
 
     /**
@@ -26,7 +16,6 @@ public function version(): string;
     public function init();
 
     /**
-     * @param string $code
      * @return string
      */
     public function execute(string $code);

src/Loaders/LoaderInterface.php

@@ -8,7 +8,7 @@ class PimcoreLoader extends BaseLoader
 {
     public function __construct(string $path)
     {
-        require_once $path . '/vendor/autoload.php';
+        require_once $path.'/vendor/autoload.php';
 
         if (class_exists('\Pimcore\Bootstrap')) {
             \Pimcore\Bootstrap::setProjectRoot();
@@ -29,6 +29,6 @@ public function version(): string
 
     public static function supports(string $path): bool
     {
-        return file_exists($path . '/vendor/autoload.php') && file_exists($path . '/vendor/pimcore/pimcore');
+        return file_exists($path.'/vendor/autoload.php') && file_exists($path.'/vendor/pimcore/pimcore');
     }
 }