Skip to content

Commit f2462c1

Browse files
authored
Merge pull request #266 from Staffbase/improve-default-messages
Improve wording of success and fail messages
2 parents 743b103 + b88f90a commit f2462c1

File tree

1 file changed

+6
-4
lines changed

1 file changed

+6
-4
lines changed

src/utils.ts

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -43,15 +43,17 @@ const appendMagicString = (comment: string): string =>
4343

4444
const commentSuccess = (owner: string, repo: string, pulls: Pull[]): string =>
4545
`
46-
🟢 Sucessfully deployed to dev.
47-
The following Pull Requests have been deployed to dev:
46+
🟢 Sucessfully merged into the dev branch.
47+
It can take up to a few minutes until the changes are rolled out to the dev system.
48+
The following Pull Requests are merged into the dev branch:
4849
${pulls.map(pull => `- ${pullURL(owner, repo, pull.number)}`).join('\n')}
4950
`
5051

5152
const commentFail = (): string =>
5253
`
53-
🚨 Unable to deploy this Pull Request to dev.
54-
Please check the logs of the github action. The Pull requests with dev-labels might have merge conflicts.
54+
🚨 Unable to merge this branch into the dev branch.
55+
This usually means that one of the PRs with a dev label has merge conflicts.
56+
Please check the logs of the github action.
5557
`
5658

5759
const pullURL = (owner: string, repo: string, number: number): string =>

0 commit comments

Comments
 (0)