fix(core): fix bug that app calling notifiers without any logs

async3619 · Dec 12, 2022 · 40bfdf5 · 40bfdf5
1 parent 5f310a7
commit 40bfdf5
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/app.ts b/src/app.ts
@@ -238,6 +238,10 @@ export class App extends Loggable {
             return;
         }
 
+        if (newLogs.length <= 0) {
+            return;
+        }
+
         const watcherMap = this.config.watcherMap;
         for (const [, notifier] of notifiers) {
             await notifier.notify(

diff --git a/src/notifiers/discord.ts b/src/notifiers/discord.ts
@@ -45,10 +45,6 @@ export class DiscordNotifier extends BaseNotifier<"Discord"> {
             throw new Error("DiscordNotifier is not initialized");
         }
 
-        if (pairs.length <= 0) {
-            return;
-        }
-
         const data: DiscordWebhookData = {
             content: null,
             embeds: [