Commit 49b84aa5 authored by Imre Farkas's avatar Imre Farkas

Merge branch 'fix-ruby-2.7-warnings-in-app-services' into 'master'

Fix Ruby 2.7 warnings in app/services

See merge request gitlab-org/gitlab!44480
parents d247ae6d c1a2a7c6
...@@ -33,7 +33,7 @@ module Ci ...@@ -33,7 +33,7 @@ module Ci
pipeline_params.fetch(:target_revision)) pipeline_params.fetch(:target_revision))
downstream_pipeline = service.execute( downstream_pipeline = service.execute(
pipeline_params.fetch(:source), pipeline_params[:execute_params]) do |pipeline| pipeline_params.fetch(:source), **pipeline_params[:execute_params]) do |pipeline|
pipeline.variables.build(@bridge.downstream_variables) pipeline.variables.build(@bridge.downstream_variables)
end end
......
...@@ -70,7 +70,7 @@ module Ci ...@@ -70,7 +70,7 @@ module Ci
push_options: params[:push_options] || {}, push_options: params[:push_options] || {},
chat_data: params[:chat_data], chat_data: params[:chat_data],
bridge: bridge, bridge: bridge,
**extra_options(options)) **extra_options(**options))
# Ensure we never persist the pipeline when dry_run: true # Ensure we never persist the pipeline when dry_run: true
@pipeline.readonly! if command.dry_run? @pipeline.readonly! if command.dry_run?
......
...@@ -13,8 +13,8 @@ module NotificationRecipients ...@@ -13,8 +13,8 @@ module NotificationRecipients
NotificationRecipient.new(user, *args).notifiable? NotificationRecipient.new(user, *args).notifiable?
end end
def self.build_recipients(*args) def self.build_recipients(target, current_user, **args)
::NotificationRecipients::Builder::Default.new(*args).notification_recipients ::NotificationRecipients::Builder::Default.new(target, current_user, **args).notification_recipients
end end
def self.build_new_note_recipients(*args) def self.build_new_note_recipients(*args)
...@@ -25,8 +25,8 @@ module NotificationRecipients ...@@ -25,8 +25,8 @@ module NotificationRecipients
::NotificationRecipients::Builder::MergeRequestUnmergeable.new(*args).notification_recipients ::NotificationRecipients::Builder::MergeRequestUnmergeable.new(*args).notification_recipients
end end
def self.build_project_maintainers_recipients(*args) def self.build_project_maintainers_recipients(target, **args)
::NotificationRecipients::Builder::ProjectMaintainers.new(*args).notification_recipients ::NotificationRecipients::Builder::ProjectMaintainers.new(target, **args).notification_recipients
end end
def self.build_new_release_recipients(*args) def self.build_new_release_recipients(*args)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment