Merge pull request #1463 from nf-core/oncomplete-fix-minimal

robsyme · web-flow · commit 674645ea6f31 · 2024-12-09T11:57:11.000-05:00
Move channel operations outside of the onComplete() block
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -16,6 +16,7 @@ Special thanks to the following for their contributions to the release:
 
 - [PR #1369](https://github.com/nf-core/rnaseq/pull/1369) - Add umicollapse as an alternative to umi-tools
 - [PR #1461](https://github.com/nf-core/rnaseq/pull/1461) - Add FASTQ linting during preprocessing
+- [PR #1463](https://github.com/nf-core/rnaseq/pull/1463) - Move channel operations outside of the onComplete() block
 
 ### Software dependencies
 
diff --git a/subworkflows/local/utils_nfcore_rnaseq_pipeline/main.nf b/subworkflows/local/utils_nfcore_rnaseq_pipeline/main.nf
@@ -116,6 +116,8 @@ workflow PIPELINE_COMPLETION {
             id, status -> pass_strand_check[id] = status
         }
 
+    def multiqc_report_list = multiqc_report.toList()
+
     //
     // Completion email and summary
     //
@@ -128,7 +130,7 @@ workflow PIPELINE_COMPLETION {
                 plaintext_email,
                 outdir,
                 monochrome_logs,
-                multiqc_report.toList()
+                multiqc_report_list.getVal()
             )
         }
 

Original file line number	Diff line number	Diff line change
`@@ -116,6 +116,8 @@ workflow PIPELINE_COMPLETION {`
`116`	`116`	`id, status -> pass_strand_check[id] = status`
`117`	`117`	`}`
`118`	`118`
	`119`	`+ def multiqc_report_list = multiqc_report.toList()`
	`120`	`+`
`119`	`121`	`//`
`120`	`122`	`// Completion email and summary`
`121`	`123`	`//`
`@@ -128,7 +130,7 @@ workflow PIPELINE_COMPLETION {`
`128`	`130`	`plaintext_email,`
`129`	`131`	`outdir,`
`130`	`132`	`monochrome_logs,`
`131`		`- multiqc_report.toList()`
	`133`	`+ multiqc_report_list.getVal()`
`132`	`134`	`)`
`133`	`135`	`}`
`134`	`136`