Switch to Laravel Pint Action for Code Styling (#151)

Author: irazasyed

.github/workflows/code-style.yml

@@ -3,23 +3,23 @@ name: Code Style
 on: [push]
 
 jobs:
-  php-cs-fixer:
+  phplint:
     if: "!contains(github.event.head_commit.message, 'skip ci')"
-
+    
     runs-on: ubuntu-latest
-
+    
     steps:
       - name: Checkout code
-        uses: actions/checkout@v2
+        uses: actions/checkout@v3
+      
+      - name: "Laravel Pint"
+        uses: aglipanci/[email protected]
         with:
-          ref: ${{ github.head_ref }}
-
-      - name: Run PHP CS Fixer
-        uses: docker://oskarstark/php-cs-fixer-ga
-
-      - name: Commit changes
+          preset: laravel
+          
+      - name: Commit changes to GitHub
         uses: stefanzweifel/git-auto-commit-action@v4
         with:
-          commit_message: Apply PHP CS Fixer changes
+          commit_message: "PHP Style Change (Laravel Pint in CI)"
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.php-cs-fixer.dist.php

@@ -17,7 +17,7 @@ class CouldNotSendNotification extends Exception
      */
     public static function telegramRespondedWithAnError(ClientException $exception): self
     {
-        if (!$exception->hasResponse()) {
+        if (! $exception->hasResponse()) {
             return new static('Telegram responded with an error but no response body found');
         }
 
@@ -43,7 +43,6 @@ public static function telegramBotTokenNotProvided(string $message): self
      * Thrown when we're unable to communicate with Telegram.
      *
      * @param $message
-     *
      * @return static
      */
     public static function couldNotCommunicateWithTelegram($message): self

src/Exceptions/CouldNotSendNotification.php

@@ -28,7 +28,7 @@ public function __construct(Dispatcher $dispatcher)
     /**
      * Send the given notification.
      *
-     * @param mixed $notifiable
+     * @param  mixed  $notifiable
      *
      * @throws CouldNotSendNotification
      */
@@ -44,7 +44,7 @@ public function send($notifiable, Notification $notification): ?array
             $to = $notifiable->routeNotificationFor('telegram', $notification)
                 ?? $notifiable->routeNotificationFor(self::class, $notification);
 
-            if (!$to) {
+            if (! $to) {
                 return null;
             }
 

src/TelegramChannel.php

@@ -44,15 +44,14 @@ public function content(string $content): self
      *
      * Generic method to attach files of any type based on API.
      *
-     * @param resource|StreamInterface|string $file
-     *
+     * @param  resource|StreamInterface|string  $file
      * @return $this
      */
     public function file($file, string $type, string $filename = null): self
     {
         $this->type = $type;
 
-        if (is_string($file) && !$this->isReadableFile($file)) {
+        if (is_string($file) && ! $this->isReadableFile($file)) {
             $this->payload[$type] = $file;
 
             return $this;

src/TelegramFile.php

@@ -13,8 +13,8 @@ class TelegramLocation extends TelegramBase implements TelegramSender
     /**
      * Telegram Location constructor.
      *
-     * @param null|float|string $latitude
-     * @param null|float|string $longitude
+     * @param  null|float|string  $latitude
+     * @param  null|float|string  $longitude
      */
     public function __construct($latitude = null, $longitude = null)
     {
@@ -24,9 +24,8 @@ public function __construct($latitude = null, $longitude = null)
     }
 
     /**
-     * @param null|float|string $latitude
-     * @param null|float|string $longitude
-     *
+     * @param  null|float|string  $latitude
+     * @param  null|float|string  $longitude
      * @return static
      */
     public static function create($latitude = null, $longitude = null): self
@@ -37,8 +36,7 @@ public static function create($latitude = null, $longitude = null): self
     /**
      * Location's latitude.
      *
-     * @param float|string $latitude
-     *
+     * @param  float|string  $latitude
      * @return $this
      */
     public function latitude($latitude): self
@@ -51,8 +49,7 @@ public function latitude($latitude): self
     /**
      * Location's latitude.
      *
-     * @param float|string $longitude
-     *
+     * @param  float|string  $longitude
      * @return $this
      */
     public function longitude($longitude): self

src/TelegramLocation.php

@@ -27,8 +27,7 @@ trait HasSharedLogic
     /**
      * Recipient's Chat ID.
      *
-     * @param int|string $chatId
-     *
+     * @param  int|string  $chatId
      * @return $this
      */
     public function to($chatId): self
@@ -121,7 +120,7 @@ public function options(array $options): self
      */
     public function toNotGiven(): bool
     {
-        return !isset($this->payload['chat_id']);
+        return ! isset($this->payload['chat_id']);
     }
 
     /**

src/Traits/HasSharedLogic.php

@@ -62,8 +62,7 @@ public function itCanSendAMessage(): void
             'parse_mode' => 'Markdown',
             'chat_id' => 12345,
         ])
-            ->andReturns(new Response(200, [], json_encode($expectedResponse)))
-        ;
+            ->andReturns(new Response(200, [], json_encode($expectedResponse)));
 
         $actualResponse = $this->channel->send(new TestNotifiable(), new TestNotification());
 
@@ -83,8 +82,7 @@ public function notificationFailedEvent(): void
 
         $this->telegram
             ->shouldReceive('sendMessage')
-            ->andThrow($exception_class, $exception_message)
-        ;
+            ->andThrow($exception_class, $exception_message);
 
         $this->dispatcher
             ->expects($this->once())
@@ -96,8 +94,7 @@ public function notificationFailedEvent(): void
                     'telegram',
                     []
                 )
-            )
-        ;
+            );
 
         $this->channel->send($notifiable, $notification);
     }

tests/TelegramChannelTest.php

@@ -15,6 +15,7 @@
 class TelegramLocationTest extends TestCase
 {
     public const TEST_LONG = -77.0364;
+
     public const TEST_LAT = 38.8951;
 
     /** @test */