summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLin Jen-Shin <godfat@godfat.org>2016-09-19 13:44:36 +0800
committerLin Jen-Shin <godfat@godfat.org>2016-10-17 07:25:20 +0000
commitbac500fdeb3ac9b29c26fd8b48a0b1974c894413 (patch)
tree3be623b8ddf311a32072b393f9485999fbd694fe
parent55f2e6ab21d4e23675fd3b1e9f5e0a5d59ea3976 (diff)
downloadgitlab-ce-bac500fdeb3ac9b29c26fd8b48a0b1974c894413.tar.gz
It no longer needs a block, feedback:
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6342#note_15434727
-rw-r--r--app/models/project_services/pipelines_email_service.rb4
1 files changed, 1 insertions, 3 deletions
diff --git a/app/models/project_services/pipelines_email_service.rb b/app/models/project_services/pipelines_email_service.rb
index b4e36a7c3de..ec27aa88f16 100644
--- a/app/models/project_services/pipelines_email_service.rb
+++ b/app/models/project_services/pipelines_email_service.rb
@@ -1,9 +1,7 @@
class PipelinesEmailService < Service
prop_accessor :recipients
boolean_accessor :notify_only_broken_pipelines
- validates :recipients,
- presence: true,
- if: ->(s) { s.activated? }
+ validates :recipients, presence: true, if: :activated?
def initialize_properties
self.properties ||= { notify_only_broken_pipelines: true }