Is this file supposed to be here? <https://github....
# team-maintainers
m
kill it with fire
e
With pleasure
Oops, in the end this is the sole remaining subworkflow (
FASTA_INDEX_DNA
) that has not been ported to nf-test. Shall we keep or remove? It's not used by any nf-core pipeline.
m
I’d rather you migrate the tests to nf-test πŸ˜…
e
Someone has to do it.. πŸ˜›
m
Might as well be you then πŸ˜‰
j
Should deprecate, not necessarily remove
someone somewhere_ might be using it...
j
e
@Simon Pearce what's the status on this one?
m
We're almost there with simon
❀️ 1
s
It is ready to merge, just needs someone to turn off the pytest tests to do so πŸ˜‰
πŸ™ 1
m
removed the requirement for pytest to pass
πŸŽ‰ 3
m
I’ve fixed the tests on the PR. CI now passes
thanks 2
Since it’s the last module with pytest, maybe someone from the core team wants to hit the Merge button ? Film themselves while doing it, and share it on Slack / at the Summit πŸ˜› ?
🀣 2