Noassignees #72
Noassignees #72
Conversation
| defp error_message(login) do | ||
| """ | ||
| @#{login} the issue doesn't have anymore assignees but is still `in-progress`. | ||
| Please assign a user to this issue or remove the `in-progress` label. |
There was a problem hiding this comment.
The message error might not be the best one, any suggestions?
There was a problem hiding this comment.
@SimonLab we should put all error messages in a central place so they can easily be edited. (doesn't have to be done now, but in future...)
For now we should update it to:
"""
:warning: @#{login} issue has **no assignee** but is still `in-progress`. Who is working on it?
Please assign a user to this issue or remove the `in-progress` label. Thanks! :heart:
"""
Codecov Report
@@ Coverage Diff @@
## master #72 +/- ##
==========================================
+ Coverage 89.28% 93.15% +3.86%
==========================================
Files 12 14 +2
Lines 56 73 +17
==========================================
+ Hits 50 68 +18
+ Misses 6 5 -1
Continue to review full report at Codecov.
|
|
At the moment the messages are linked to their rule. Agree that we need to create a config file for the rules. I'd like to allow users to create their own rule for each of their repositories so it will make sense to refactor the error message with this feature |
to be merged after #70
see #71
add rule when assignee is removed from an issue but the
in-progresslabel is still on.