Tiny CI <#C049MBCEW06|> improvments in <#C01NHS4CC...
# request-review
m
Tiny CI #C049MBCEW06 improvments in #C01NHS4CC91 before pushing it in #CGFUX04HZ and continuing experimenting before pushing upstream to #C07PTBHLK7D or even #C049MBCEW06: https://github.com/nf-core/rnavar/pull/270
✅ 1
s
interesting...can u add a test that produces warning and capture that
m
I have that in sarek
basically the way I see it, we only really need a yml file containing the params, and detailing which kind of test we want on the pipeline side
For all unit tests, plain nf-test is good
for pipeline tests, I think we need to simplify, homogenize and streamline how we test, and I quite like where I'm getting
s
sounds good
m
One thing I am unhappy for now is how Nextflow deals with stderr
most warning nextflow produces are actually stdout
but I have a discussion about this with Ben already
and I want to deal with it in a nicer way
so that's why I'm highly interested in nf-test dev
We're getting pretty good at reproducing results, I want to make sure we reproduce errors reliably over time too
s
yup sounds like a plan for the new year
m
Yeah, I was thinking waiting for January and try to organize a meeting so we can actually start doing some work
👍 1