Refactor mail notifier in Journal class

kiwilan · Feb 2, 2024 · 98cef82 · 98cef82
1 parent e2aaf34
commit 98cef82
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 8 deletions.
diff --git a/.github/workflows/run-tests.yml b/.github/workflows/run-tests.yml
@@ -38,12 +38,10 @@ jobs:
           extensions: dom, curl, libxml, mbstring, zip, pcntl, pdo, sqlite, pdo_sqlite, bcmath, soap, intl, gd, exif, iconv, imagick, fileinfo
           coverage: pcov
 
-      - uses: JarvusInnovations/background-action@v1
-        name: Setup mailpit
-        with:
-          run: |
-            sudo bash < <(curl -sL https://raw.githubusercontent.com/axllent/mailpit/develop/install.sh)
-            mailpit
+      - name: Setup mailpit
+        run: |
+          sudo bash < <(curl -sL https://raw.githubusercontent.com/axllent/mailpit/develop/install.sh)
+          mailpit
 
       - name: Setup problem matchers
         run: |

diff --git a/phpstan.neon.dist b/phpstan.neon.dist
@@ -6,7 +6,6 @@ parameters:
     paths:
         - src
         - config
-        - database
     tmpDir: build/phpstan
     checkOctaneCompatibility: true
     checkModelProperties: true

diff --git a/src/Journal.php b/src/Journal.php
@@ -110,7 +110,7 @@ public function toDatabase(Model|Authenticatable|Collection|array|null $users =
     public function notifier(string $type): self
     {
         $this->notifier = match ($type) {
-            'mail' => Notifier::mail()->auto(),
+            'mail' => Notifier::mail(),
             'slack' => Notifier::slack(config('steward.slack.webhook')),
             'discord' => Notifier::discord(config('steward.discord.webhook')),
             default => null,