Stage pr created to slack stage (#2572)

* Send slack messages only once & improvements

* Don't go into the catch block for slack messages

.github/workflows/merge-to-main.yaml

@@ -9,7 +9,6 @@ on:
 
 env:
   MILO_RELEASE_SLACK_WH: ${{ secrets.MILO_RELEASE_SLACK_WH }}
-  MILO_STAGE_SLACK_WH: ${{secrets.MILO_STAGE_SLACK_WH}}
 
 jobs:
   merge-to-main:

.github/workflows/merge-to-stage.js

@@ -132,7 +132,7 @@ const merge = async ({ prs, type }) => {
     try {
       if (files.some((file) => SEEN[file])) {
         commentOnPR(
-          `Skipped ${number}: ${title} due to file overlap. Merging will be attempted in the next batch`,
+          `Skipped ${number}: ${title} due to file "${file}" overlap. Merging will be attempted in the next batch`,
           number
         );
         continue;
@@ -157,10 +157,11 @@ const merge = async ({ prs, type }) => {
           number,
           title,
           prefix,
-        })
+        }).catch(console.error)
       );
       await new Promise((resolve) => setTimeout(resolve, 5000));
     } catch (error) {
+      files.forEach((file) => (SEEN[file] = false));
       commentOnPR(`Error merging ${number}: ${title} ` + error.message, number);
     }
   }

.github/workflows/merge-to-stage.yaml

@@ -11,6 +11,7 @@ env:
   MILO_RELEASE_SLACK_WH: ${{ secrets.MILO_RELEASE_SLACK_WH }}
   REQUIRED_APPROVALS: ${{ secrets.REQUIRED_APPROVALS }}
   SLACK_HIGH_IMPACT_PR_WEBHOOK: ${{ secrets.SLACK_HIGH_IMPACT_PR_WEBHOOK }}
+  MILO_STAGE_SLACK_WH: ${{secrets.MILO_STAGE_SLACK_WH}}
 
 jobs:
   merge-to-stage: