Commit f2ed30f8 authored by Rémy Coutable's avatar Rémy Coutable

Don't pass `env` anymore to GitAccess, ChangeAccess, and ForcePush

Signed-off-by: default avatarRémy Coutable <remy@rymai.me>
parent f39d4b6a
......@@ -7,7 +7,7 @@ module Gitlab
attr_reader :user_access, :project, :skip_authorization, :protocol
def initialize(
change, user_access:, project:, env: {}, skip_authorization: false,
change, user_access:, project:, skip_authorization: false,
protocol:
)
@oldrev, @newrev, @ref = change.values_at(:oldrev, :newrev, :ref)
......@@ -15,7 +15,6 @@ module Gitlab
@tag_name = Gitlab::Git.tag_name(@ref)
@user_access = user_access
@project = project
@env = env
@skip_authorization = skip_authorization
@protocol = protocol
end
......@@ -99,7 +98,7 @@ module Gitlab
end
def forced_push?
Gitlab::Checks::ForcePush.force_push?(@project, @oldrev, @newrev, env: @env)
Gitlab::Checks::ForcePush.force_push?(@project, @oldrev, @newrev)
end
def update?
......
module Gitlab
module Checks
class ForcePush
def self.force_push?(project, oldrev, newrev, env: {})
def self.force_push?(project, oldrev, newrev)
return false if project.empty_repo?
# Created or deleted branch
......
......@@ -20,13 +20,12 @@ module Gitlab
attr_reader :actor, :project, :protocol, :user_access, :authentication_abilities
def initialize(actor, project, protocol, authentication_abilities:, env: {})
def initialize(actor, project, protocol, authentication_abilities:)
@actor = actor
@project = project
@protocol = protocol
@authentication_abilities = authentication_abilities
@user_access = UserAccess.new(user, project: project)
@env = env
end
def check(cmd, changes)
......@@ -191,7 +190,6 @@ module Gitlab
change,
user_access: user_access,
project: project,
env: @env,
skip_authorization: deploy_key?,
protocol: protocol
).exec
......
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