Skip to content

Commit

Permalink
chore: use correct name in pull request bot comments
Browse files Browse the repository at this point in the history
The event sender may not be the person who opened the pull request.

If another user pushes to the pull request, I think it would be better
not to change the name in the bot's comment.
  • Loading branch information
kevinbuhmann committed Jun 6, 2022
1 parent e79a480 commit 1b65708
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/bot.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ jobs:
comment-id: ${{steps.find-comment.outputs.comment-id}}
issue-number: ${{steps.get_pr_event.outputs.pullRequestNumber}}
body: |
👋 @${{github.event.sender.login}},
👋 @${{github.event.pull_request.user.login}},
* 🙏 The Clarity team thanks you for opening a pull request
* ⏳ The build for this PR has started
Expand All @@ -44,7 +44,7 @@ jobs:
comment-id: ${{steps.find-comment.outputs.comment-id}}
issue-number: ${{steps.get_pr_event.outputs.pullRequestNumber}}
body: |
👋 @${{github.event.sender.login}},
👋 @${{github.event.pull_request.user.login}},
* 🙏 The Clarity team thanks you for opening a pull request
* 🎉 The build for this PR has succeeded
Expand All @@ -63,7 +63,7 @@ jobs:
comment-id: ${{steps.find-comment.outputs.comment-id}}
issue-number: ${{steps.get_pr_event.outputs.pullRequestNumber}}
body: |
👋 @${{github.event.sender.login}},
👋 @${{github.event.pull_request.user.login}},
* 😭 The build for this PR has failed
* 🗒 Please check out the [build log](${{github.event.workflow_run.html_url}})
Expand Down

0 comments on commit 1b65708

Please sign in to comment.