Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve Zulipbot messages #216

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 1 addition & 5 deletions config/templates/abandonWarning.md
Original file line number Diff line number Diff line change
@@ -1,5 +1 @@
Hello @{assignee}, you have been unassigned from this issue because you have not updated this issue or any referenced pull requests for over {total} days.

You can reclaim this issue or claim any other issue by commenting `@{username} claim` on that issue.

Thanks for your contributions, and hope to see you again soon!
@{assignee} You have been unassigned from this issue because you have not made any updates for over {total} days. Please feel free to reclaim the issue if you decide to pick up again. Thanks!
13 changes: 6 additions & 7 deletions config/templates/contributorAddition.md
Original file line number Diff line number Diff line change
@@ -1,10 +1,9 @@
Welcome to Zulip, @{commenter}! We just sent you an invite to collaborate on this repository at https://github.com/{repoOwner}/{repoName}/invitations. Please accept this invite in order to claim this issue and begin a fun, rewarding experience contributing to Zulip!
Welcome to Zulip, @{commenter}! To get started, please take the following three steps **in order**:

Here's some tips to get you off to a good start:
1. Complete all the steps in the [Your first codebase contribution](https://zulip.readthedocs.io/en/latest/overview/contributing.html#your-first-codebase-contribution) section of the [Zulip contributor guide](https://zulip.readthedocs.io/en/latest/contributing/index.html) up to "Claiming an issue".
2. Post a comment here describing your plan for how to approach this issue. Before you claim an issue, you should be confident that you will be able to tackle it effectively.
3. Accept the invite to collaborate on this repository at https://github.com/{repoOwner}/{repoName}/invitations. To avoid filling your inbox, we recommend [unwatching this repository](https://help.github.com/articles/unwatching-repositories/).

- Join me on the [Zulip developers' server](https://chat.zulip.org), to get help, chat about this issue, and meet the other developers.
- [Unwatch this repository](https://help.github.com/articles/unwatching-repositories/), so that you don't get 100 emails a day.
The issue will be assigned to you when all three steps have been completed.

As you work on this issue, you'll also want to refer to the [Zulip code contribution guide](https://zulip.readthedocs.io/en/latest/contributing/index.html), as well as the rest of the developer documentation on that site.

See you on the other side (that is, the pull request side)!
If someone else claims this issue while you're working through the checklist, no worries! You can contribute by providing feedback on their pull request. If you’ve made good progress in understanding part of the codebase, you can also find another “help wanted” issue in the same area to work on.
4 changes: 2 additions & 2 deletions config/templates/fixCommitWarning.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
Hello @{author}, it seems like you have referenced #{issues} in your pull request description, but you have not referenced {issuePronoun} in your commit message description(s). When you reference an issue in a commit message, it automatically closes the corresponding issue when the commit is merged.
Hello @{author}, it seems like you have referenced #{issues} in your pull request description, but you have not referenced {issuePronoun} in your commit message description(s). Referencing an issue in a commit message automatically closes the corresponding issue when the commit is merged, which makes the issue tracker easier to manage.

Please run `git commit --amend` in your command line client to amend your commit message description with `Fixes #{fixIssues}.`.

Expand All @@ -14,6 +14,6 @@ Date: Sat Mar 18 13:42:40 2017 -0700
Fixes #51.
```

Thank you for your contributions to Zulip!
To learn how to write a great commit message, please refer to our [guide](https://zulip.readthedocs.io/en/latest/contributing/version-control.html#commit-messages).

<!-- fixCommitWarning -->
8 changes: 2 additions & 6 deletions config/templates/inactiveWarning.md
Original file line number Diff line number Diff line change
@@ -1,9 +1,5 @@
Hello @{assignee}, you claimed this issue to work on it, but this issue and any referenced pull requests haven't been updated for {remind} days. Are you still working on this issue?
@{assignee} We noticed that you have not made any updates to this issue or linked PRs for {remind} days. Please comment here if you are still actively working on it. Otherwise, we'd appreciate a quick `@{username} abandon` comment so that someone else can claim this issue and continue from where you left off.

If so, please update this issue by leaving a comment on this issue to let me know that you're still working on it. Otherwise, I'll automatically remove you from this issue in {abandon} days.

If you've decided to work on something else, simply comment `@{username} abandon` so that someone else can claim it and continue from where you left off.

Thank you for your valuable contributions to Zulip!
If we don't hear back, you will be automatically unassigned in {abandon} days. Thanks!

<!-- inactiveWarning -->
4 changes: 1 addition & 3 deletions config/templates/inviteError.md
Original file line number Diff line number Diff line change
@@ -1,3 +1 @@
Hello @{commenter}, it looks like we've already sent you a collaboration invite at https://github.com/{repoOwner}/{repoName}/invitations, but you haven't accepted it yet!

Please accept the invite and try to claim this issue again afterwards. We look forward to your contributions!
@{commenter} It looks like we've already sent you a collaboration invite at https://github.com/{repoOwner}/{repoName}/invitations, but you haven't accepted it yet. Please accept the invite, and then try to claim this issue again. Thanks!
4 changes: 1 addition & 3 deletions config/templates/multipleClaimWarning.md
Original file line number Diff line number Diff line change
@@ -1,3 +1 @@
Hello @{commenter}, it looks like someone has already claimed this issue! Since we believe multiple assignments to the same issue may cause some confusion, we encourage you to search for other unclaimed issues to work on. However, you can always reclaim this issue if no one is working on it.

We look forward to your valuable contributions!
@{commenter} This issue cannot be claimed, as someone else is already working on it. Please see our [contributor guide](https://zulip.readthedocs.io/en/latest/overview/contributing.html#your-first-codebase-contribution) for advice on finding an issue to work on. Thanks!
4 changes: 1 addition & 3 deletions config/templates/needsReviewWarning.md
Original file line number Diff line number Diff line change
@@ -1,3 +1 @@
Hello @{assignees}, @{author} updated this pull request over {days} days ago in response to your review, but you haven't checked back in on their changes! Please leave some feedback for them or mark this pull request as ready for merging.

Thank you for reviewing this pull request!
Hello @{assignees}, @{author} updated this pull request over {days} days ago in response to your review. Please leave some feedback on next steps for this PR. Thanks!
4 changes: 1 addition & 3 deletions config/templates/updateWarning.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
Hello @{author}, a Zulip maintainer reviewed this pull request over {days} days ago, but you haven't updated your pull request since. Please take a look at the requested changes and update your pull request accordingly.

Thank you for your valuable contributions to Zulip!
@{author} A Zulip maintainer reviewed this PR over {days} days ago. Please take a look at the requested changes and update your PR, or leave a comment if you will not be able to do so soon. Thanks!

<!-- updateWarning -->