Merge pull request #15 from danielma/dma/dont-notify-no-work-hours

fix(Reminder): no notifications unless someone is working
danielma · May 15, 2017 · 5fcb222 · 5fcb222
2 parents 1716006 + d3c3b0c
commit 5fcb222
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,