From 8fc4a7b539edd6d4518b2eaaced0b9e8922b93a4 Mon Sep 17 00:00:00 2001
From: Rob Syme <rob.syme@gmail.com>
Date: Thu, 5 Dec 2024 19:04:17 -0500
Subject: [PATCH] Fix bug where workflow can hang if the email parameter is
 set.

This fix moves all queue channel operations out of the onComplete block and into the main workflow.
---
 subworkflows/local/utils_nfcore_sarek_pipeline/main.nf | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/subworkflows/local/utils_nfcore_sarek_pipeline/main.nf b/subworkflows/local/utils_nfcore_sarek_pipeline/main.nf
index f4983278a1..ce568284c7 100644
--- a/subworkflows/local/utils_nfcore_sarek_pipeline/main.nf
+++ b/subworkflows/local/utils_nfcore_sarek_pipeline/main.nf
@@ -183,6 +183,8 @@ workflow PIPELINE_COMPLETION {
     main:
     summary_params = paramsSummaryMap(workflow, parameters_schema: "nextflow_schema.json")
 
+    def multiqc_report_list = multiqc_report.toList()
+
     //
     // Completion email and summary
     //
@@ -195,7 +197,7 @@ workflow PIPELINE_COMPLETION {
                 plaintext_email,
                 outdir,
                 monochrome_logs,
-                multiqc_report.toList()
+                multiqc_report_list.getVal()
             )
         }