Add linting check to project (#10)

* Add laravel pint

* Add workflow for linting

Author: ingalless

.github/workflows/lint.yml

@@ -0,0 +1,22 @@
+name: Lint
+
+on:
+  pull_request:
+    branches: [ "main" ]
+
+jobs:
+  lint:
+    runs-on: ubuntu-latest
+
+    steps:
+    - uses: shivammathur/setup-php@15c43e89cdef867065b0213be354c2841860869e
+      with:
+        php-version: '8.0'
+
+    - uses: actions/checkout@v3
+
+    - name: Install Dependencies
+      run: composer install -q --no-ansi --no-interaction --no-scripts --no-progress --prefer-dist
+
+    - name: Check Code Against Linter
+      run: composer lint

composer.json

@@ -25,6 +25,10 @@
         "guzzlehttp/guzzle": "^7.2",
         "laravel/framework": "^9.0"
     },
+    "scripts": {
+        "lint": "vendor/bin/pint --test",
+        "lint:fix": "vendor/bin/pint" 
+    },
     "extra": {
         "laravel": {
             "providers": [
@@ -41,5 +45,8 @@
         "allow-plugins": {
             "kylekatarnls/update-helper": false
         }
+    },
+    "require-dev": {
+        "laravel/pint": "^1.0"
     }
 }

src/Transport.php

@@ -13,6 +13,7 @@ class Transport extends AbstractTransport
 {
     /**
      * Graph api configuration
+     *
      * @var array
      */
     private array $config;
@@ -23,16 +24,15 @@ public function __construct(array $config)
         $this->config = $config;
     }
 
-
     protected function doSend(SentMessage $message): void
     {
         $token = $this->getToken();
         $email = MessageConverter::toEmail($message->getOriginalMessage());
         $url = sprintf('https://graph.microsoft.com/v1.0/users/%s/sendMail', $email->getFrom()[0]->getEncodedAddress());
         $response = Http::withHeaders([
-            'Authorization' => sprintf('Bearer %s', $token)
+            'Authorization' => sprintf('Bearer %s', $token),
         ])->post($url, [
-            "message" => $this->getMessage($email)
+            'message' => $this->getMessage($email),
         ]);
         $response->throw();
     }
@@ -44,7 +44,7 @@ public function getToken()
             'client_id' => $this->config['client_id'],
             'client_secret' => $this->config['client_secret'],
             'scope' => 'https://graph.microsoft.com/.default',
-            'grant_type' => 'client_credentials'
+            'grant_type' => 'client_credentials',
         ]);
         $response->throw();
 
@@ -62,18 +62,18 @@ public function __toString(): string
     private function getMessage(Email $email)
     {
         return array_filter([
-            "from" => $this->getRecipient($email->getFrom()[0]),
-            "sender" => $this->getRecipient($email->getFrom()[0]),
-            "toRecipients" => $this->getRecipientsCollection($email->getTo()),
-            "ccRecipients" => $this->getRecipientsCollection($email->getCc()),
-            "bccRecipients" => $this->getRecipientsCollection($email->getBcc()),
-            "replyTo" => $this->getRecipientsCollection($email->getReplyTo()),
-            "subject" => $email->getSubject(),
-            "body" => [
-                "contentType" => $email->getTextBody() ? 'Text' : 'HTML',
-                "content" => $email->getTextBody() ?? $email->getHtmlBody(),
+            'from' => $this->getRecipient($email->getFrom()[0]),
+            'sender' => $this->getRecipient($email->getFrom()[0]),
+            'toRecipients' => $this->getRecipientsCollection($email->getTo()),
+            'ccRecipients' => $this->getRecipientsCollection($email->getCc()),
+            'bccRecipients' => $this->getRecipientsCollection($email->getBcc()),
+            'replyTo' => $this->getRecipientsCollection($email->getReplyTo()),
+            'subject' => $email->getSubject(),
+            'body' => [
+                'contentType' => $email->getTextBody() ? 'Text' : 'HTML',
+                'content' => $email->getTextBody() ?? $email->getHtmlBody(),
             ],
-            "attachments" => $this->getAttachmentsCollection($email->getAttachments())
+            'attachments' => $this->getAttachmentsCollection($email->getAttachments()),
         ]);
     }
 
@@ -91,7 +91,7 @@ private function getRecipient($address): array
             'emailAddress' => array_filter([
                 'address' => $address->getAddress(),
                 'name' => $address->getName(),
-            ])
+            ]),
         ];
     }
 
@@ -106,10 +106,10 @@ private function getAttachmentsCollection($attachments)
     private function getAttachment(DataPart $attachment)
     {
         return array_filter([
-            "@odata.type" => "#microsoft.graph.fileAttachment",
-            "name" => $attachment->getName() ?? $attachment->getFilename(),
-            "contentType" => $attachment->getContentType(),
-            "contentBytes" => base64_encode($attachment->getBody()),
+            '@odata.type' => '#microsoft.graph.fileAttachment',
+            'name' => $attachment->getName() ?? $attachment->getFilename(),
+            'contentType' => $attachment->getContentType(),
+            'contentBytes' => base64_encode($attachment->getBody()),
         ]);
     }
 }