Apply suggestions from code review by @drpatelh

Co-authored-by: Harshil Patel <drpatelh@users.noreply.github.com>

software/multiqc/main.nf

@@ -5,28 +5,28 @@ params.options = [:]
 def options    = initOptions(params.options)
 
 process MULTIQC {
-    tag "multiqc"
     label 'process_medium'
     publishDir "${params.outdir}",
         mode: params.publish_dir_mode,
         saveAs: { filename -> saveFiles(filename: filename, options: params.options, publish_dir: getSoftwareName(task.process), publish_id: '') }
 
-    conda(params.enable_conda ? "bioconda::multiqc=1.9" : null)
+    conda     (params.enable_conda ? "bioconda::multiqc=1.9" : null)
     container "quay.io/biocontainers/multiqc:1.9--pyh9f0ad1d_0"
 
     input:
-    path multiqc_config
+    path multiqc_files
-    path multiqc_custom_config
-    path('fastqc/*')
 
     output:
     path "*multiqc_report.html", emit: report
-    path "*_data", emit: data
+    path "*_data"              , emit: data
-    path "*_plots", optional: true, emit: plots
+    path "*_plots"             , optional:true, emit: plots
+    path "*.version.txt"       , emit: version
 
     script:
+    def software = getSoftwareName(task.process)
     def custom_config = params.multiqc_config ? "--config $multiqc_custom_config" : ''
     """
     multiqc -f $options.args $custom_config .
+     multiqc --version | sed -e "s/multiqc, version //g" > ${software}.version.txt
     """
 }

software/multiqc/meta.yml

@@ -28,16 +28,16 @@ params:
             Run the module with Conda using the software specified
             via the `conda` directive
 input:
-    - generic_report:
+    - multiqc_files:
         type: file
         description: |
-            List of report, for example the html and zip output of FastQC
+            List of reports / files recognised by MultiQC, for example the html and zip output of FastQC
 output:
-    - dir:
+    - data:
         type: dir
         description: MultiQC data dir
         pattern: "multiqc_data"
-    - html:
+    - report:
         type: file
         description: MultiQC report file
         pattern: "multiqc_report.html"

tests/software/multiqc/main.nf

@@ -6,15 +6,13 @@ include { MULTIQC } from '../../../software/multiqc/main.nf' addParams(options:
 include { test_paired_end } from '../fastqc/main.nf' addParams(options: [publish_dir: 'test_paired_end'])
 
 ch_multiqc_config = file("$launchDir/tests/software/multiqc/multiqc_config.yml", checkIfExists: true)
-ch_multiqc_custom_config = params.multiqc_config ? Channel.fromPath(params.multiqc_config) : Channel.empty()
 
 workflow test_multiqc {
     test_paired_end()
 
     input = [
-        ch_multiqc_config,
+        ch_multiqc_config
-        ch_multiqc_custom_config.collect().ifEmpty([]),
+            .mix(test_paired_end.out.zip.collect { it[1] }.ifEmpty([]))
-        test_paired_end.out.zip.collect { it[1] }.ifEmpty([])
     ]
 
     MULTIQC(*input)