Merge pull request #5 from binafy/add-pint-action

[1.x] Add Pint action

.github/workflows/pint.yml

@@ -0,0 +1,34 @@
+name: Fix Code Style
+
+on: [push, pull_request]
+
+jobs:
+    lint:
+        runs-on: ubuntu-latest
+        strategy:
+            fail-fast: true
+            matrix:
+                php: [8.3]
+
+        steps:
+            - name: Checkout code
+              uses: actions/checkout@v4
+
+            - name: Setup PHP
+              uses: shivammathur/setup-php@v2
+              with:
+                  php-version: ${{ matrix.php }}
+                  extensions: json, dom, curl, libxml, mbstring
+                  coverage: none
+
+            - name: Install Pint
+              run: composer global require laravel/pint
+
+            - name: Run Pint
+              run: pint
+
+            - name: Commit linted files
+              uses: stefanzweifel/git-auto-commit-action@v5
+              with:
+                  commit_message: "Fixes coding style"
+

composer.json

@@ -49,7 +49,8 @@
     },
     "require-dev": {
         "pestphp/pest-plugin-laravel": "^1.4.0|^2.0.0",
-        "orchestra/testbench": "^7.0|^8.0|^9.0"
+        "orchestra/testbench": "^7.0|^8.0|^9.0",
+        "laravel/pint": "^1.5"
     },
     "config": {
         "sort-packages": true,

src/Models/Cart.php

@@ -47,7 +47,7 @@ public function scopeFirstOrCreateWithStoreItems(
         Builder $query,
         Model $item,
         int $quantity = 1,
-        int|null $userId = null
+        ?int $userId = null
     ): Builder {
         if (is_null($userId)) {
             $userId = auth()->id();

src/Providers/LaravelCartServiceProvider.php

@@ -11,8 +11,8 @@ class LaravelCartServiceProvider extends ServiceProvider
      */
     public function register(): void
     {
-        $this->loadMigrationsFrom(__DIR__ . '/../../database/migrations');
-        $this->mergeConfigFrom(__DIR__ . '/../../config/laravel-cart.php', 'laravel-cart');
+        $this->loadMigrationsFrom(__DIR__.'/../../database/migrations');
+        $this->mergeConfigFrom(__DIR__.'/../../config/laravel-cart.php', 'laravel-cart');
     }
 
     /**
@@ -22,12 +22,12 @@ public function boot(): void
     {
         // Publish Config
         $this->publishes([
-            __DIR__ . '/../../config/laravel-cart.php' => config_path('laravel-cart.php'),
+            __DIR__.'/../../config/laravel-cart.php' => config_path('laravel-cart.php'),
         ], 'laravel-cart-config');
 
         // Publish Migrations
         $this->publishes([
-            __DIR__ . '/../../database/migrations' => database_path('migrations'),
+            __DIR__.'/../../database/migrations' => database_path('migrations'),
         ], 'laravel-cart-migrations');
     }
 }

tests/Feature/CartDeleteTest.php

@@ -1,14 +1,11 @@
 <?php
 
 use Binafy\LaravelCart\Models\Cart;
-use Binafy\LaravelCart\Models\CartItem;
 use Illuminate\Foundation\Testing\RefreshDatabase;
-use Illuminate\Support\Facades\Artisan;
 use Tests\SetUp\Models\Product;
 use Tests\SetUp\Models\User;
+
 use function Pest\Laravel\assertDatabaseCount;
-use function Pest\Laravel\assertDatabaseHas;
-use function PHPUnit\Framework\assertInstanceOf;
 
 /*
  * Use `RefreshDatabase` for delete migration data for each test.
@@ -59,7 +56,7 @@
     assertDatabaseCount('cart_items', 2);
 });
 
-test('can empty the cart', function() {
+test('can empty the cart', function () {
     $user = User::query()->create(['name' => 'Milwad', 'email' => 'milwad.dev@gmail.comd']);
     $product1 = Product::query()->create(['title' => 'Product 1']);
     $product2 = Product::query()->create(['title' => 'Product 2']);

tests/Feature/CartStoreTest.php

@@ -6,6 +6,7 @@
 use Illuminate\Support\Facades\Artisan;
 use Tests\SetUp\Models\Product;
 use Tests\SetUp\Models\User;
+
 use function Pest\Laravel\assertDatabaseCount;
 use function Pest\Laravel\assertDatabaseHas;
 use function PHPUnit\Framework\assertInstanceOf;

tests/SetUp/Models/User.php

@@ -2,7 +2,6 @@
 
 namespace Tests\SetUp\Models;
 
-
 use Illuminate\Foundation\Auth\User as Authenticatable;
 
 class User extends Authenticatable

tests/TestCase.php

@@ -12,7 +12,7 @@ abstract class TestCase extends \Orchestra\Testbench\TestCase
     /**
      * Load package service provider.
      *
-     * @param \Illuminate\Foundation\Application $app
+     * @param  \Illuminate\Foundation\Application  $app
      */
     protected function getPackageProviders($app): array
     {
@@ -22,16 +22,16 @@ protected function getPackageProviders($app): array
     /**
      * Define environment setup.
      *
-     * @param \Illuminate\Foundation\Application $app
+     * @param  \Illuminate\Foundation\Application  $app
      */
     protected function getEnvironmentSetUp($app): void
     {
         // Set default database to use sqlite :memory:
         $app['config']->set('database.default', 'testing');
         $app['config']->set('database.connections.testing', [
-            'driver'   => 'sqlite',
+            'driver' => 'sqlite',
             'database' => ':memory:',
-            'prefix'   => '',
+            'prefix' => '',
         ]);
 
         // Set app key