Template update, modules and a few bug fixes: <htt...
# release-review-trading
d
Template update, modules and a few bug fixes: https://github.com/nf-core/metatdenovo/pull/444 (I'm afraid not as easy as my recent phyloplace release.) I have limited time this week, but happy to trade if you can wait until next week. My PR is certainly not urgent either.
👀 1
✅ 1
c
I am planning a release update next week, I will ping you 🙂
d
Perfect! Thanks!
j
@Daniel Lundin #C02K5GX2W93 is still looking for s econd if you wuould like to trade?
d
Absolutely
I changed my plans a bit, so I think I can look at #C02K5GX2W93 today
j
❤️
I'll look at metatdenovo now
@Daniel Lundin why are there so many local modules in this one 🫠
A lot of them look like they would be perfectly fine as proper module, or even amodule exists?
d
I was a bit afraid of that question. And I don't have a better answer than the first virtue. On a more serious note, I actually looked through this during this cycle but I didn't come any further than this. I will keep thinking about it
j
mmmm, how desperate are yo ufor this to be released? 😬
d
Quite urgent I'd say. There is an important bug fix in this and I have major work for #C01U1E95JQ5 planned that I'd like to start soon
j
Ok, then I'll allow this but I'm going to make an issue blocking the next release until this is fixed 😉 It looks like you've spent a long time updating all these local modules for somethign that should have just been
nf-core modules update
and done. It's meaning this is quite a painful review for me too 😅 (normally I can ignore most nf-core moduel files)
d
Thanks
j
Approved with a few minor comments
d
thanks! Will turn to #C02K5GX2W93 after my meeting, so around noon
j
The said issue: https://github.com/nf-core/metatdenovo/issues/445 But like I said you can proceed with 1.4 for now
👍 1
d
Good that you remind me about
DIAMOND_BLASTP
. I remember I did a local because at the time I couldn't get
nf-core modules patch
to work. It turned out it stumbled on Vim temporary files. Since then I've patched many modules successfully.
👍 1