diff options
author | Richard Macklin <richard.github@nrm.com> | 2017-02-03 21:01:45 -0800 |
---|---|---|
committer | Richard Macklin <richard.github@nrm.com> | 2017-02-03 21:01:45 -0800 |
commit | 7cd260b10727cef0621ecef429ec89e00873b1b7 (patch) | |
tree | fc3d7974bd39fdc90f16c7e2f8d738c43fc15b11 /app/services/notification_service.rb | |
parent | 0e2c96e709161b35daeae2dad3aefcf9c85a75aa (diff) | |
download | gitlab-ce-7cd260b10727cef0621ecef429ec89e00873b1b7.tar.gz |
Refactor NotificationService#pipeline_finished to use skip_current_user
instead of passing nil for current_user
Diffstat (limited to 'app/services/notification_service.rb')
-rw-r--r-- | app/services/notification_service.rb | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/app/services/notification_service.rb b/app/services/notification_service.rb index 5a7d5ef8747..e5283720913 100644 --- a/app/services/notification_service.rb +++ b/app/services/notification_service.rb @@ -327,8 +327,9 @@ class NotificationService recipients ||= build_recipients( pipeline, pipeline.project, - nil, # The acting user, who won't be added to recipients - action: pipeline.status).map(&:notification_email) + pipeline.user, + action: pipeline.status, + skip_current_user: false).map(&:notification_email) if recipients.any? mailer.public_send(email_template, pipeline, recipients).deliver_later |