Merge pull request #97 from TheDragonCode/1.x

Added ignoring mentions from Dependabot

Author: andrey-helldar

app/Data/ItemData.php

@@ -6,19 +6,20 @@
 
 class ItemData extends Data
 {
-    public bool $isOpen;
+    public ?int $ownerId = null;
 
-    public bool $isMerged;
+    public bool $isOpen = false;
+
+    public bool $isMerged = true;
 
     public function __construct(?array $data)
     {
         if (is_null($data)) {
-            $this->isOpen   = false;
-            $this->isMerged = true;
-
             return;
         }
 
+        $this->ownerId = $this->get($data, 'user.id');
+
         $this->isOpen   = $this->get($data, 'state') === 'open';
         $this->isMerged = (bool) $this->get($data, 'merged');
     }

app/Services/GitHub.php

@@ -181,7 +181,7 @@ protected function shouldSkip(NotificationData $notification, ItemData $item): b
             return true;
         }
 
-        if ($this->exceptMentions && $notification->reason === 'mention') {
+        if ($this->exceptMentions && $notification->reason === 'mention' && $this->isNotDependabot($item)) {
             return true;
         }
 
@@ -216,4 +216,9 @@ protected function detected(int $count): void
 
         Output::info("unread $pluralized detected", $count);
     }
+
+    protected function isNotDependabot(ItemData $item): bool
+    {
+        return $item->ownerId !== config('bots.dependabot');
+    }
 }

config/bots.php

@@ -0,0 +1,5 @@
+<?php
+
+return [
+    'dependabot' => 49699333,
+];