diff options
| author | Lin Jen-Shin <godfat@godfat.org> | 2016-09-15 00:00:02 +0800 |
|---|---|---|
| committer | Lin Jen-Shin <godfat@godfat.org> | 2016-09-15 00:00:02 +0800 |
| commit | 844d4e24af244b34d425e4d027a8acfe235b7178 (patch) | |
| tree | f14a862c6988c2eea793b0b5657925bc290121e2 /app/models/project_services/pipelines_email_service.rb | |
| parent | 1b8bbe5cd0c3f0ed66fb0b393d0dab7c85495a0b (diff) | |
| download | gitlab-ce-844d4e24af244b34d425e4d027a8acfe235b7178.tar.gz | |
Use keyword arg, feedback:
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6019/diffs#note_15336212
Diffstat (limited to 'app/models/project_services/pipelines_email_service.rb')
| -rw-r--r-- | app/models/project_services/pipelines_email_service.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/app/models/project_services/pipelines_email_service.rb b/app/models/project_services/pipelines_email_service.rb index 5f8d7554057..e133b1696da 100644 --- a/app/models/project_services/pipelines_email_service.rb +++ b/app/models/project_services/pipelines_email_service.rb @@ -26,7 +26,7 @@ class PipelinesEmailService < Service %w[pipeline] end - def execute(data, force = false) + def execute(data, force: false) return unless supported_events.include?(data[:object_kind]) return unless force || should_pipeline_be_notified?(data) @@ -66,7 +66,7 @@ class PipelinesEmailService < Service end def test(data) - result = execute(data, true) + result = execute(data, force: true) { success: true, result: result } rescue StandardError => error |