#91 Wrong wording of MR comment announcement

Open
opened 6 years ago by fr33domlover · 2 comments
22:50 <fpbot> fr33domlover commented on issue #88 funbot | Thank you :) https://notabug.org/fr33domlover/funbot/pulls/88#issuecomment-7471

It says commented on issue, but it's really a comment on merge request 88. Look at the code that figures out the web hooks and formats web listener events, and fix this.

22:50 <fpbot> fr33domlover commented on issue #88 funbot | Thank you :) https://notabug.org/fr33domlover/funbot/pulls/88#issuecomment-7471 It says commented on issue, but it's really a comment on *merge request* 88. Look at the code that figures out the web hooks and formats web listener events, and fix this.

Should we call it a merge request (phrasing I prefer) or pull request (phrasing gogs uses)?

Should we call it a merge request (phrasing I prefer) or pull request (phrasing gogs uses)?
fr33domlover commented 6 years ago
Owner

In the announcements the bot makes, it already uses the term MR. So that's the term to use when fixing this issue :)

In the announcements the bot makes, it already uses the term *MR*. So that's the term to use when fixing this issue :)
Sign in to join this conversation.
No Milestone
No assignee
2 Participants
Loading...
Cancel
Save
There is no content yet.