fix(Reminder): no notifications unless someone is working

If it's not a work hour for _any_ of the suggested reviewers, don't bother sending a notification Fixes #14
danielma · May 15, 2017 · d3c3b0c · d3c3b0c
1 parent 1716006
commit d3c3b0c
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/lib/review_bot/reminder.rb b/lib/review_bot/reminder.rb
@@ -74,7 +74,7 @@ def potential_notifications
           pull.reviewers.include?(reviewer['github'])
         end
 
-        next if suggested_reviewers.empty?
+        next if suggested_reviewers.select(&:work_hour?).empty?
 
         Notification.new(
           pull_request: pull,