Merge pull request #11586 from dmarcoux/ignore-closed-merged-reopened…

…-PRs-MRs Ignore temporarily closed, merged and reopened PRs/MRs
openSUSE · Sep 7, 2021 · c5ad511 · c5ad511
2 parents 60a7a30 + 3ed7226
commit c5ad511
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 0 deletions.
diff --git a/src/api/app/models/workflow/step/branch_package_step.rb b/src/api/app/models/workflow/step/branch_package_step.rb
@@ -4,6 +4,9 @@ class Workflow::Step::BranchPackageStep < ::Workflow::Step
   def call(options = {})
     return unless valid?
 
+    # FIXME: Support closed/merged/reopened PRs
+    return if scm_webhook.closed_merged_pull_request? || scm_webhook.reopened_pull_request?
+
     branched_package = find_or_create_branched_package
 
     add_or_update_branch_request_file(package: branched_package)

diff --git a/src/api/app/models/workflow/step/configure_repositories.rb b/src/api/app/models/workflow/step/configure_repositories.rb
@@ -9,6 +9,9 @@ class Workflow::Step::ConfigureRepositories < Workflow::Step
   def call(_options = {})
     return unless valid?
 
+    # FIXME: Support closed/merged/reopened PRs
+    return if scm_webhook.closed_merged_pull_request? || scm_webhook.reopened_pull_request?
+
     step_instructions[:repositories].each do |repository_instructions|
       repository = Repository.includes(:architectures).find_or_create_by(name: repository_instructions[:name], project: Project.get_by_name(target_project_name))
 

diff --git a/src/api/app/models/workflow/step/link_package_step.rb b/src/api/app/models/workflow/step/link_package_step.rb
@@ -4,6 +4,9 @@ class Workflow::Step::LinkPackageStep < ::Workflow::Step
   def call(options = {})
     return unless valid?
 
+    # FIXME: Support closed/merged/reopened PRs
+    return if scm_webhook.closed_merged_pull_request? || scm_webhook.reopened_pull_request?
+
     workflow_filters = options.fetch(:workflow_filters, {})
 
     # Updated PR