Merge pull request #1320 from argos-ci/clarify-status-messages

feat: clarify status messages
argos-ci · Jun 7, 2024 · 8971f22 · 8971f22
2 parents f182ff8 + 286ea52
commit 8971f22
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/apps/backend/src/build-notification/notification.ts b/apps/backend/src/build-notification/notification.ts
@@ -100,19 +100,19 @@ async function getNotificationDescription(input: {
       const statsMessage = await getStatsMessage(buildId);
       if (!statsMessage) {
         if (isReference) {
-          return "Used as comparison baseline";
+          return "Used as comparison baseline, no changes found";
         }
         return "Everything's good!";
       }
       if (isReference) {
-        return `${statsMessage} — used as comparison baseline`;
+        return `Used a comparison baseline, no changes found — ${statsMessage}`;
       }
-      return `${statsMessage} — no change`;
+      return `${statsMessage} — no changes found`;
     }
     case "diff-detected": {
       const statsMessage = await getStatsMessage(buildId);
       if (isReference) {
-        return `${statsMessage} — used as comparison baseline`;
+        return `${statsMessage}, automatically approved and used as comparison baseline`;
       }
       return `${statsMessage} — waiting for your decision`;
     }