updates #62 to check if there are any reviewers before adding a no as…

…signee warning #128

web/controllers/rules/pr/no_assignee.ex

@@ -7,7 +7,8 @@ defmodule Dwylbot.Rules.PR.NoAssignee do
   @rule_name "pr_no_assignee"
 
   def apply?(payload) do
-    payload["action"] in ~w(labeled unassigned assigned)
+    reviewers = payload["pull_request"]["requested_reviewers"]  |> Enum.map(&(&1["login"]))
+    payload["action"] in ~w(labeled unassigned assigned) && Enum.empty?(reviewers)
   end
 
   def check(payload, _get_data?, token) do
@@ -37,7 +38,7 @@ defmodule Dwylbot.Rules.PR.NoAssignee do
 
   defp error_message(login) do
     """
-    :warning: @#{login}, the pull request is in "awaiting-review" but doesn't have a correct assignee.
+    :warning: @#{login}, the pull request is in "awaiting-review" but doesn't have a reviewer or a correct assignee.
     Please assign someone to review the pull request, thanks.
     """
   end