style: add pint and apply styles

Author: lindyhopchris

.github/workflows/tests.yml

@@ -40,5 +40,8 @@ jobs:
         max_attempts: 5
         command: composer update --prefer-dist --no-interaction --no-progress
 
+    - name: Execute style
+      run: composer run style
+
     - name: Execute tests
-      run: vendor/bin/phpunit
+      run: composer run test

composer.json

@@ -27,6 +27,7 @@
     "require-dev": {
         "laravel/cashier": "^15.6",
         "laravel/legacy-factories": "^1.4",
+        "laravel/pint": "^1.24",
         "orchestra/testbench": "^9.14|^10.0",
         "phpunit/phpunit": "^11.5"
     },
@@ -40,6 +41,15 @@
             "CloudCreativity\\LaravelStripe\\Tests\\": "tests/lib"
         }
     },
+    "scripts": {
+        "all": [
+            "@style",
+            "@test"
+        ],
+        "style": "pint --test",
+        "style:fix": "pint",
+        "test": "phpunit"
+    },
     "extra": {
         "branch-alias": {
             "dev-develop": "1.x-dev"

config/stripe.php

@@ -1,6 +1,7 @@
 <?php
+
 /**
- * This file is part of cloudcreativity/laravel-stripe
+ * This file is part of cloudcreativity/laravel-stripe.
  *
  * (c) Christopher Gammie <[email protected]>
  *

database/factories/StripeAccountFactory.php

@@ -1,4 +1,5 @@
 <?php
+
 /*
  * Copyright 2023 Cloud Creativity Limited
  *

database/factories/StripeEventFactory.php

@@ -1,4 +1,5 @@
 <?php
+
 /*
  * Copyright 2023 Cloud Creativity Limited
  *

database/migrations/2019_07_17_074500_create_stripe_accounts_and_events.php

@@ -1,4 +1,5 @@
 <?php
+
 /*
  * Copyright 2023 Cloud Creativity Limited
  *
@@ -22,7 +23,6 @@
 
 class CreateStripeAccountsAndEvents extends Migration
 {
-
     /**
      * Run the migration.
      *

pint.json

@@ -0,0 +1,37 @@
+{
+    "preset": "psr12",
+    "rules": {
+        "array_syntax": true,
+        "assign_null_coalescing_to_coalesce_equal": true,
+        "clean_namespace": true,
+        "declare_strict_types": false,
+        "list_syntax": true,
+        "no_empty_phpdoc": true,
+        "no_superfluous_elseif": true,
+        "no_superfluous_phpdoc_tags": {
+            "remove_inheritdoc": true
+        },
+        "no_unneeded_braces": true,
+        "no_useless_nullsafe_operator": true,
+        "no_whitespace_before_comma_in_array": true,
+        "normalize_index_brace": true,
+        "nullable_type_declaration": true,
+        "nullable_type_declaration_for_default_null_value": true,
+        "ordered_attributes": true,
+        "ordered_traits": true,
+        "ordered_types": {
+            "null_adjustment": "always_last"
+        },
+        "phpdoc_summary": true,
+        "phpdoc_trim": true,
+        "phpdoc_types": true,
+        "phpdoc_types_order": {
+            "null_adjustment": "always_last"
+        },
+        "simple_to_complex_string_variable": true,
+        "ternary_to_null_coalescing": true,
+        "trailing_comma_in_multiline": {
+            "elements": ["arguments", "arrays", "match", "parameters"]
+        }
+    }
+}

src/Assert.php

@@ -1,4 +1,5 @@
 <?php
+
 /*
  * Copyright 2023 Cloud Creativity Limited
  *
@@ -42,7 +43,6 @@ public static function id($expected, $id)
     /**
      * Assert that the currency is supported by the application.
      *
-     * @param $currency
      * @return void
      */
     public static function supportedCurrency($currency)
@@ -59,8 +59,6 @@ public static function supportedCurrency($currency)
     /**
      * Assert that the currency and amount are chargeable.
      *
-     * @param $currency
-     * @param $amount
      * @return void
      * @see https://stripe.com/docs/currencies#minimum-and-maximum-charge-amounts
      */
@@ -82,7 +80,6 @@ public static function chargeAmount($currency, $amount)
     /**
      * Assert that the value is a zero-decimal amount.
      *
-     * @param $amount
      * @return void
      * @see https://stripe.com/docs/currencies#zero-decimal
      */

src/Client.php

@@ -1,4 +1,5 @@
 <?php
+
 /*
  * Copyright 2023 Cloud Creativity Limited
  *
@@ -25,16 +26,13 @@
 
 class Client
 {
-
     /**
      * @var Dispatcher
      */
     private $events;
 
     /**
      * Client constructor.
-     *
-     * @param Dispatcher $events
      */
     public function __construct(Dispatcher $events)
     {
@@ -44,8 +42,6 @@ public function __construct(Dispatcher $events)
     /**
      * @param string $class
      * @param string $method
-     * @param mixed ...$args
-     * @return mixed
      */
     public function __invoke($class, $method, ...$args)
     {
@@ -66,11 +62,6 @@ public function __invoke($class, $method, ...$args)
 
     /**
      * Execute the static Stripe call.
-     *
-     * @param $class
-     * @param $method
-     * @param array $args
-     * @return mixed
      */
     protected function execute($class, $method, array $args)
     {

src/Config.php

@@ -1,4 +1,5 @@
 <?php
+
 /*
  * Copyright 2023 Cloud Creativity Limited
  *
@@ -28,7 +29,6 @@
 
 class Config
 {
-
     /**
      * Get the Stripe API Key.
      *
@@ -65,13 +65,13 @@ public static function clientId()
     /**
      * Get the Connect account model.
      *
-     * @return StripeAccount|mixed
+     * @return mixed|StripeAccount
      */
     public static function connectModel()
     {
         $class = self::fqn('connect.model');
 
-        return new $class;
+        return new $class();
     }
 
     /**
@@ -83,7 +83,7 @@ public static function connectOwner()
     {
         $class = self::fqn('connect.owner');
 
-        return new $class;
+        return new $class();
     }
 
     /**
@@ -118,13 +118,13 @@ public static function connectErrorView()
     }
 
     /**
-     * @return StripeEvent|mixed
+     * @return mixed|StripeEvent
      */
     public static function webhookModel()
     {
         $class = self::fqn('webhooks.model');
 
-        return new $class;
+        return new $class();
     }
 
     /**
@@ -139,7 +139,7 @@ public static function webhookSigningSecrect($name)
             throw new RuntimeException("Webhook signing secret does not exist: {$name}");
         }
 
-        if (!is_string($secret)|| empty($secret)) {
+        if (!is_string($secret) || empty($secret)) {
             throw new RuntimeException("Invalid webhook signing secret: {$name}");
         }
 
@@ -168,7 +168,7 @@ public static function webhookQueue($type, $connect = false)
         $path = sprintf(
             'webhooks.%s.%s',
             $connect ? 'connect' : 'account',
-            str_replace('.', '_', $type)
+            str_replace('.', '_', $type),
         );
 
         return array_replace([
@@ -242,7 +242,6 @@ public static function logExclude()
     /**
      * @param string $key
      * @param mixed|null $default
-     * @return mixed
      */
     private static function get($key, $default = null)
     {
@@ -265,7 +264,6 @@ private static function fqn($key)
     }
 
     /**
-     * @param $key
      * @return bool
      */
     private static function has($key)