Merge pull request #7 from netsells/feature/php8.2

Upgrades to Laravel 10 and PHP 8.2, plus fixes from static analysis and code styles

Author: Jakub Gawron

.github/workflows/phpcsfixer.yml

@@ -0,0 +1,26 @@
+name: Format with PHP CS Fixer
+
+on: [pull_request, workflow_dispatch]
+
+jobs:
+  php-cs-fixer:
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: Setup PHP
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: '8.2'
+          extensions: zip
+          coverage: none
+
+      - name: Install Dependencies
+        run: composer install --no-ansi --no-interaction --no-scripts --prefer-dist
+
+      - run: composer run format
+
+      - uses: stefanzweifel/git-auto-commit-action@v4
+        with:
+          commit_message: Apply php-cs-fixer changes

.github/workflows/phpstan.yml

@@ -0,0 +1,22 @@
+name: Perform static analysis with PHPStan
+
+on: [pull_request, workflow_dispatch]
+
+jobs:
+  phpstan:
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: Setup PHP
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: '8.2'
+          extensions: zip
+          coverage: none
+
+      - name: Install Dependencies
+        run: composer install --no-ansi --no-interaction --no-scripts --prefer-dist
+
+      - run: composer run phpstan

.github/workflows/tests.yml

@@ -4,19 +4,14 @@ on: [pull_request, workflow_dispatch]
 
 jobs:
   test:
-    # not self-hosted, because it's a public repo
     runs-on: ubuntu-latest
 
-    # we want to run it on combination of Laravel 8.12+ and PHP 8+
+    # we want to run it on supported combination of Laravel and PHP versions
     strategy:
       fail-fast: false
       matrix:
-        php: ['8.0', '8.1']
-        laravel: ['^8.12', '^9.0', '^10.0']
-        exclude:
-          # minimum php version for Laravel 10 is PHP 8.1 so we need to exclude (php 8.0 and laravel 10) combination
-          - php: '8.0'
-            laravel: '^10.0'
+        php: ['8.1', '8.2']
+        laravel: ['^9.0', '^10.0']
 
     steps:
       - name: Checkout the repo
@@ -35,4 +30,4 @@ jobs:
         run: composer require laravel/framework $LARAVEL_VERSION --no-interaction --no-scripts --prefer-dist
 
       - name: Execute tests
-        run: vendor/bin/phpunit
+        run: composer run test

.gitignore

@@ -2,4 +2,5 @@
 composer.lock
 .idea
 .phpunit.result.cache
-.phpunit.cache
+.phpunit.cache
+.php-cs-fixer.cache

.php-cs-fixer.php

@@ -0,0 +1,25 @@
+<?php
+
+use PhpCsFixer\Config;
+use PhpCsFixer\Finder;
+
+/** @var Config $config */
+$config = require_once join(DIRECTORY_SEPARATOR, [__DIR__, 'vendor', 'netsells', 'code-standards-laravel', 'phpcsfixer', 'config.php']);
+
+$rules = array_merge_recursive($config->getRules(), [
+    // set project specific rules here
+]);
+
+$finder = Finder::create()
+    ->in([
+        __DIR__ . '/src',
+        __DIR__ . '/tests',
+    ])
+    ->name('*.php')
+    ->notName('*.blade.php')
+    ->ignoreDotFiles(true)
+    ->ignoreVCS(true);
+
+return $config
+    ->setFinder($finder)
+    ->setRules($rules);

composer.json

@@ -20,18 +20,25 @@
         }
     ],
     "require": {
-        "php": ">=8.0",
+        "php": "^8.1 || ^8.2",
         "ext-json": "*",
-        "laravel/framework": "^8.12 || ^9.0 || ^10.0"
+        "laravel/framework": "^9.0 || ^10.0"
     },
     "require-dev": {
-        "orchestra/testbench": "^6.0 || ^7.0 || ^8.0",
-        "tmarsteel/mockery-callable-mock": "~2.1"
+        "orchestra/testbench": "^7.0 || ^8.0",
+        "tmarsteel/mockery-callable-mock": "~2.1",
+        "netsells/code-standards-laravel": "^1.1"
     },
     "autoload": {
         "psr-4": {
             "Netsells\\Http\\Resources\\": "src/",
             "Netsells\\Http\\Resources\\Tests\\": "tests/"
         }
+    },
+    "scripts": {
+        "phpstan": "vendor/bin/phpstan",
+        "lint": "vendor/bin/php-cs-fixer fix --dry-run --diff",
+        "format": "vendor/bin/php-cs-fixer fix",
+        "test": "vendor/bin/phpunit"
     }
 }

phpstan.neon

@@ -0,0 +1,15 @@
+includes:
+    - ./vendor/netsells/code-standards-laravel/phpstan/extension.neon
+
+parameters:
+    paths:
+        - src
+        - tests
+
+    # The level 9 is the highest level
+    level: 5
+
+    ignoreErrors:
+    excludePaths:
+
+    checkMissingIterableValueType: false

src/Eloquent/EloquentDeferredValue.php

@@ -2,6 +2,7 @@
 
 namespace Netsells\Http\Resources\Eloquent;
 
+use Illuminate\Support\Collection;
 use Netsells\Http\Resources\DeferredValue;
 use Netsells\Http\Resources\Json\JsonResource;
 use Netsells\Http\Resources\Json\ResourceCollection;
@@ -26,12 +27,12 @@ public static function resolve(array $deferredValues): void
     {
         static::loadEloquentRelations($deferredValues);
 
-        collect($deferredValues)
+        Collection::make($deferredValues)
             ->filter(function (EloquentDeferredValue $deferredValue) {
-                return $deferredValue->resolver;
+                return (bool) $deferredValue->resolver;
             })
             ->each(function (EloquentDeferredValue $deferredValue) {
-                $relations = collect($deferredValue->relations)->map(function ($relation) use ($deferredValue) {
+                $relations = Collection::make($deferredValue->relations)->map(function ($relation) use ($deferredValue) {
                     return data_get($deferredValue->resource, $relation);
                 })->all();
 
@@ -41,6 +42,7 @@ public static function resolve(array $deferredValues): void
 
     /**
      * Begins eager loading eloquent model relations.
+     *
      * @param static[] $deferredValues
      */
     abstract protected static function loadEloquentRelations(array $deferredValues): void;

src/Eloquent/Load.php

@@ -8,11 +8,11 @@
 class Load extends EloquentDeferredValue
 {
     /**
-     * @inheritDoc
+     * {@inheritDoc}
      */
     protected static function loadEloquentRelations(array $deferredValues): void
     {
-        collect($deferredValues)->groupBy('relations')
+        Collection::make($deferredValues)->groupBy('relations')
             ->each(function (Collection $collection, $relation) {
                 EloquentCollection::make($collection->pluck('resource.resource'))
                     ->load($relation);

src/Eloquent/LoadCount.php

@@ -8,11 +8,11 @@
 class LoadCount extends EloquentDeferredValue
 {
     /**
-     * @inheritDoc
+     * {@inheritDoc}
      */
     protected static function loadEloquentRelations(array $deferredValues): void
     {
-        collect($deferredValues)->groupBy('relations')
+        Collection::make($deferredValues)->groupBy('relations')
             ->each(function (Collection $collection, $relation) {
                 EloquentCollection::make($collection->pluck('resource.resource'))
                     ->loadCount($relation);