Skip to content

Commit

Permalink
Wording changes
Browse files Browse the repository at this point in the history
  • Loading branch information
mokimo committed May 8, 2024
1 parent af90aa6 commit f929f77
Showing 1 changed file with 8 additions and 3 deletions.
11 changes: 8 additions & 3 deletions .github/workflows/merge-to-stage.js
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ const SLACK = {
fileOverlap: ({ html_url, number, title }) =>
`:fast_forward: Skipping <${html_url}|${number}: ${title}> due to overlap in files.`,
merge: ({ html_url, number, title, highImpact }) =>
`:merged:${highImpact} Stage merge PR <${html_url}|${number}: ${title}>.`,
`:merged:${highImpact} PR merged to stage: <${html_url}|${number}: ${title}>.`,
failingChecks: ({ html_url, number, title }) =>
`:x: Skipping <${html_url}|${number}: ${title}> due to failing checks`,
requireApprovals: ({ html_url, number, title }) =>
Expand Down Expand Up @@ -145,7 +145,12 @@ const merge = async ({ prs }) => {
const isHighImpact = labels.includes(LABELS.highImpact);
if (isHighImpact && process.env.SLACK_HIGH_IMPACT_PR_WEBHOOK) {
await slackNotification(
SLACK.merge({ html_url, number, title, highImpact: ' :alert:' }),
SLACK.merge({
html_url,
number,
title,
highImpact: ' :alert: High impact',
}),
process.env.SLACK_HIGH_IMPACT_PR_WEBHOOK
);
}
Expand All @@ -154,7 +159,7 @@ const merge = async ({ prs }) => {
html_url,
number,
title,
highImpact: isHighImpact ? ' :alert:' : '',
highImpact: isHighImpact ? ' :alert: High impact' : '',
})
);
}
Expand Down

0 comments on commit f929f77

Please sign in to comment.