search

nf-core / demultiplex

Demultiplexing pipeline for sequencing data
https://nf-co.re/demultiplex
MIT License
41 stars 36 forks source link

Include test of PE data #189

Closed k1sauce closed 1 month ago

k1sauce commented 3 months ago

Description of feature

There is some logic around handling of PE and SE fastq files, would be nice to have a test for this as well.

grst commented 1 month ago

As discussed with @atrigila and @nschcolnicov fixing this would entail adding a test case to https://github.com/nf-core/demultiplex/tree/master/tests/pipeline that runs as part of the CI.

nschcolnicov commented 1 month ago

Addressing this issue in this PR: https://github.com/nf-core/demultiplex/pull/214