From e9e2577465b26ac3a2c3be5b530cc455601806be Mon Sep 17 00:00:00 2001 From: Martin Cavoj Date: Wed, 17 Dec 2025 13:07:42 +0100 Subject: [PATCH] Remove unused _build parameter from pipeline_execution_policy_build? The `_build` parameter is no longer used in the `pipeline_execution_policy_build?` method. Remove it to clean up the code. Update call site to not pass argument to pipeline_execution_policy_build? --- ee/lib/ee/gitlab/ci/pipeline/chain/set_build_sources.rb | 2 +- lib/gitlab/ci/pipeline/chain/set_build_sources.rb | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/ee/lib/ee/gitlab/ci/pipeline/chain/set_build_sources.rb b/ee/lib/ee/gitlab/ci/pipeline/chain/set_build_sources.rb index 3e41880f25212b..fe2b4698303878 100644 --- a/ee/lib/ee/gitlab/ci/pipeline/chain/set_build_sources.rb +++ b/ee/lib/ee/gitlab/ci/pipeline/chain/set_build_sources.rb @@ -9,7 +9,7 @@ module SetBuildSources extend ::Gitlab::Utils::Override override :pipeline_execution_policy_build? - def pipeline_execution_policy_build?(_build) + def pipeline_execution_policy_build? command.pipeline_policy_context.pipeline_execution_context.creating_policy_pipeline? end diff --git a/lib/gitlab/ci/pipeline/chain/set_build_sources.rb b/lib/gitlab/ci/pipeline/chain/set_build_sources.rb index 1c0f386d90b033..3ae961b5b95d47 100644 --- a/lib/gitlab/ci/pipeline/chain/set_build_sources.rb +++ b/lib/gitlab/ci/pipeline/chain/set_build_sources.rb @@ -8,7 +8,7 @@ class SetBuildSources < Chain::Base def perform! command.pipeline_seed.stages.each do |stage| stage.statuses.each do |job| - job_source = if pipeline_execution_policy_build?(job) + job_source = if pipeline_execution_policy_build? 'pipeline_execution_policy' elsif scan_execution_policy_build?(job) 'scan_execution_policy' @@ -28,7 +28,7 @@ def break? private # Overridden in EE - def pipeline_execution_policy_build?(_build) + def pipeline_execution_policy_build? false end -- GitLab