Add Laravel pint and run (#241)

* Add Laravel pint and run

* StyleCI

* Lower pint version requirement

* Add pint lint workflow from core

* Remove .styleCI.yaml

Author: ryanmitchell

.github/workflows/pint-lint.yml

@@ -0,0 +1,20 @@
+name: Lint PHP code style issues
+
+on:
+  pull_request:
+    paths:
+      - '**.php'
+
+jobs:
+  lint-php-code-styling:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v3
+
+      - name: Check PHP code style issues
+        uses: aglipanci/[email protected]
+        with:
+          testMode: true
+          verboseMode: true

.styleci.yml

@@ -29,6 +29,7 @@
     },
     "require-dev": {
         "doctrine/dbal": "^3.3",
+        "laravel/pint": "^1.0",
         "orchestra/testbench": "^7.0 || ^8.0",
         "phpunit/phpunit": "^9.4"
     },

composer.json

@@ -75,11 +75,11 @@ public function metaExists()
     {
         return Blink::once('eloquent-asset-meta-exists-'.$this->id(), function () {
             return app('statamic.eloquent.assets.model')::query()
-            ->where([
-                'container' => $this->containerHandle(),
-                'folder' => $this->folder(),
-                'basename' => $this->basename(),
-            ])->count() > 0;
+                ->where([
+                    'container' => $this->containerHandle(),
+                    'folder' => $this->folder(),
+                    'basename' => $this->basename(),
+                ])->count() > 0;
         });
     }
 

src/Assets/Asset.php

@@ -36,7 +36,7 @@ public function findByHandle(string $handle): ?AssetContainerContract
         });
     }
 
-    public function make(string $handle = null): AssetContainerContract
+    public function make(?string $handle = null): AssetContainerContract
     {
         return app(AssetContainerContract::class)->handle($handle);
     }

src/Assets/AssetContainerRepository.php

@@ -22,9 +22,6 @@ public function getConnection()
 
     /**
      * Prefixes table if defined.
-     *
-     * @param  string  $table
-     * @return string
      */
     protected function prefix(string $table): string
     {

src/Database/BaseMigration.php

@@ -13,7 +13,7 @@ public static function bindings(): array
     {
         return [
             EntryContract::class => app('statamic.eloquent.entries.entry'),
-            QueryBuilder::class  => EntryQueryBuilder::class,
+            QueryBuilder::class => EntryQueryBuilder::class,
         ];
     }
 
@@ -33,7 +33,7 @@ public function find($id): ?EntryContract
         return $this->substitutionsById[$item->id()] ?? $item;
     }
 
-    public function findByUri(string $uri, string $site = null): ?EntryContract
+    public function findByUri(string $uri, ?string $site = null): ?EntryContract
     {
         $blinkKey = 'eloquent-entry-'.md5(urlencode($uri)).($site ? '-'.$site : '');
         $item = Blink::once($blinkKey, function () use ($uri, $site) {

src/Entries/EntryRepository.php

@@ -13,9 +13,9 @@ public function all(): VariablesCollection
     {
         return VariablesCollection::make(
             VariablesModel::all()
-            ->each(function ($model) {
-                return app(Variables::class)::fromModel($model);
-            })
+                ->each(function ($model) {
+                    return app(Variables::class)::fromModel($model);
+                })
         );
     }
 

src/Globals/GlobalVariablesRepository.php

@@ -46,7 +46,7 @@ public function findByHandle($handle): ?TaxonomyContract
         return $taxonomyModel ? app(TaxonomyContract::class)->fromModel($taxonomyModel) : null;
     }
 
-    public function findByUri(string $uri, string $site = null): ?Taxonomy
+    public function findByUri(string $uri, ?string $site = null): ?Taxonomy
     {
         $collection = Facades\Collection::all()
             ->first(function ($collection) use ($uri, $site) {

src/Taxonomies/TaxonomyRepository.php

@@ -40,7 +40,7 @@ public function find($id): ?TermContract
         return $term;
     }
 
-    public function findByUri(string $uri, string $site = null): ?TermContract
+    public function findByUri(string $uri, ?string $site = null): ?TermContract
     {
         $site = $site ?? $this->stache->sites()->first();