Use Pint instead of StyleCI

.github/workflows/pint.yml

@@ -0,0 +1,23 @@
+name: PHP Linting (Pint)
+on:
+    workflow_dispatch:
+    push:
+        branches-ignore:
+            - "dependabot/npm_and_yarn/*"
+jobs:
+    phplint:
+        runs-on: ubuntu-latest
+        steps:
+            - uses: actions/checkout@v3
+              with:
+                  fetch-depth: 2
+            - name: "laravel-pint"
+              uses: aglipanci/laravel-pint-action@0.1.0
+              with:
+                  preset: laravel
+
+            - name: Commit changes
+              uses: stefanzweifel/git-auto-commit-action@v4
+              with:
+                  commit_message: PHP Linting (Pint)
+                  skip_fetch: true

.github/workflows/run-tests.yml

@@ -0,0 +1,60 @@
+name: "Run Tests - Current"
+
+on: [push, pull_request]
+
+jobs:
+    test:
+        runs-on: ubuntu-latest
+        strategy:
+            fail-fast: false
+            matrix:
+                php: [8.3, 8.2, 8.1, 8.0, 7.4]
+                laravel: [9.*, 8.*, 10.*]
+                dependency-version: [prefer-lowest, prefer-stable]
+                exclude:
+                    - laravel: 10.*
+                      php: 8.0
+                    - laravel: 10.*
+                      php: 7.4
+                    - laravel: 9.*
+                      php: 7.4
+                    - laravel: 8.*
+                      php: 8.1
+                    - laravel: 8.*
+                      php: 8.2
+                    - laravel: 8.*
+                      php: 8.3
+                include:
+                    - laravel: 10.*
+                      testbench: 8.*
+                    - laravel: 9.*
+                      testbench: 7.*
+                    - laravel: 8.*
+                      testbench: 6.*
+
+        name: P${{ matrix.php }} - L${{ matrix.laravel }} - ${{ matrix.dependency-version }}
+
+        steps:
+            - name: Checkout code
+              uses: actions/checkout@v4
+
+            - name: Cache dependencies
+              uses: actions/cache@v3
+              with:
+                  path: ~/.composer/cache/files
+                  key: dependencies-laravel-${{ matrix.laravel }}-php-${{ matrix.php }}-composer-${{ hashFiles('composer.json') }}
+
+            - name: Setup PHP
+              uses: shivammathur/setup-php@v2
+              with:
+                  php-version: ${{ matrix.php }}
+                  extensions: curl, mbstring, zip, pcntl, pdo, sqlite, pdo_sqlite, iconv
+                  coverage: pcov
+
+            - name: Install dependencies
+              run: |
+                  composer require "laravel/framework:${{ matrix.laravel }}" "orchestra/testbench:${{ matrix.testbench }}" "symfony/console:>=4.3.4" "mockery/mockery:^1.3.2" --no-interaction --no-update
+                  composer update --${{ matrix.dependency-version }} --prefer-dist --no-interaction
+                  cp .env.example .env
+            - name: Execute tests
+              run: vendor/bin/phpunit

.gitignore

@@ -8,3 +8,4 @@ coverage
 .phpunit.result.cache
 .DS_Store
 .idea
+.phpunit.cache

composer.json

@@ -16,7 +16,7 @@
         }
     ],
     "require": {
-        "php": "^7.3 || ^8.0",
+        "php": "^7.4 || ^8.0 || ^8.1 || ^8.2 || ^8.3",
         "guzzlehttp/guzzle": "~6.0 || ~7.0",
         "illuminate/support": "5.7.* || 5.8.* ||^6.0 || ^7.0 || ^8.0 || ^9.0 || ^10.0",
         "symfony/dotenv": "^4.2 || ^5.1"
@@ -27,7 +27,7 @@
         "phpstan/extension-installer": "^1.1",
         "phpstan/phpstan-deprecation-rules": "^1.0",
         "phpstan/phpstan-phpunit": "^1.0",
-        "phpunit/phpunit": "^8.4 || ^9.3.3"
+        "phpunit/phpunit": "^8.4 || ^9.3.3 || ^10.0"
     },
     "autoload": {
         "psr-4": {
@@ -43,12 +43,12 @@
         "analyse": "vendor/bin/phpstan analyse",
         "test": "vendor/bin/phpunit",
         "test-coverage": "vendor/bin/phpunit --coverage-html coverage"
-
     },
     "config": {
         "sort-packages": true,
         "allow-plugins": {
-            "phpstan/extension-installer": true
+            "phpstan/extension-installer": true,
+            "pestphp/pest-plugin": false
         }
     },
     "extra": {

phpunit.xml.dist

@@ -1,21 +1,6 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<phpunit
-    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-    bootstrap="tests/autoload.php"
-    backupGlobals="false"
-    backupStaticAttributes="false"
-    colors="true"
-    verbose="true"
-    convertErrorsToExceptions="true"
-    convertNoticesToExceptions="true"
-    convertWarningsToExceptions="true"
-    processIsolation="false"
-    stopOnFailure="false"
-    xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/9.3/phpunit.xsd">
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" bootstrap="tests/autoload.php" backupGlobals="false" colors="true" processIsolation="false" stopOnFailure="false" xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.5/phpunit.xsd" cacheDirectory=".phpunit.cache" backupStaticProperties="false">
   <coverage>
-    <include>
-      <directory suffix=".php">src/</directory>
-    </include>
     <report>
       <clover outputFile="build/logs/clover.xml"/>
       <html outputDirectory="build/coverage"/>
@@ -30,4 +15,9 @@
       <directory suffix="Test.php">./tests/Feature</directory>
     </testsuite>
   </testsuites>
+  <source>
+    <include>
+      <directory suffix=".php">src/</directory>
+    </include>
+  </source>
 </phpunit>

src/AirtableManager.php

@@ -35,8 +35,6 @@ public function __construct($app)
 
     /**
      * Get a airtable table instance.
-     *
-     * @param  string  $table
      */
     public function table(string $table)
     {

src/Api/AirtableApiClient.php

@@ -11,16 +11,22 @@ class AirtableApiClient implements ApiClient
     private $client;
 
     private $typecast;
+
     private $base;
+
     private $table;
+
     private $delay;
 
     private $filters = [];
+
     private $fields = [];
+
     private $sorts = [];
+
     private $offset = false;
 
-    public function __construct($base, $table, $access_token, Http $client = null, $typecast = false, $delayBetweenRequests = 200000)
+    public function __construct($base, $table, $access_token, ?Http $client = null, $typecast = false, $delayBetweenRequests = 200000)
     {
         $this->base = $base;
         $this->table = $table;

src/Api/ApiClient.php

@@ -4,7 +4,7 @@
 
 interface ApiClient
 {
-    public function get(string $id = null);
+    public function get(?string $id = null);
 
     public function post($contents = null);
 

tests/Feature/ClientTest.php

@@ -20,8 +20,8 @@ public function it_can_be_instantiated()
     public function it_can_post()
     {
         $expectedResponse = [
-            'id'          => 'randomlygenerated',
-            'fields'      => ['Company Name' => 'Tapp Network'],
+            'id' => 'randomlygenerated',
+            'fields' => ['Company Name' => 'Tapp Network'],
             'createdTime' => 'timestamp',
         ];
 
@@ -63,13 +63,13 @@ public function it_can_sort_asc()
         //Ascending sort
         $expectedResponseAsc = [
             [
-                'id'          => 0,
-                'fields'      => ['Company Name' => 'A Network'],
+                'id' => 0,
+                'fields' => ['Company Name' => 'A Network'],
                 'createdTime' => 'timestamp',
             ],
             [
-                'id'          => 1,
-                'fields'      => ['Company Name' => 'B Network'],
+                'id' => 1,
+                'fields' => ['Company Name' => 'B Network'],
                 'createdTime' => 'timestamp',
             ],
         ];
@@ -93,13 +93,13 @@ public function it_can_sort_desc()
         //Descending sort
         $expectedResponseDesc = [
             [
-                'id'          => 1,
-                'fields'      => ['Company Name' => 'B Network'],
+                'id' => 1,
+                'fields' => ['Company Name' => 'B Network'],
                 'createdTime' => 'timestamp',
             ],
             [
-                'id'          => 0,
-                'fields'      => ['Company Name' => 'A Network'],
+                'id' => 0,
+                'fields' => ['Company Name' => 'A Network'],
                 'createdTime' => 'timestamp',
             ],
         ];