Hello :blush: I have attended to all (human and bo...
# multiqc
k
Hello 😊 I have attended to all (human and bot) reviewer comments in this PR: github.com/MultiQC/MultiQC/pull/3612 and submitted two separate PRs to fix mypy issues, separate from business logic: github.com/MultiQC/MultiQC/pull/3627 and github.com/MultiQC/MultiQC/pull/3613. 🙏🤓
p
Thank you!
I was away on vacation until last week and am now swamped with other work, so won't have a chance to get to MultiQC PRs for a little while yet sorry
It is on my todo list though, I can't promise when, but I will definitely get to yours and get them merged and released as soon as I can 🙂
🙏 1
I know it's frustrating to sit and wait for your PR to get merged sorry 😞 But without any funding for MultiQC it's basically a personal project of mine now so it has to fit in in the gaps around my other work.
💯 1
k
Thats not a complaint at all @Phil Ewels - and thanks for all the work! I just am getting into the nf-core community things and got an idea that people mention things like that in the channels 😇 So I used a chance to give a little context.
🙏🏻 1
just a gentle monthly reminder about these PRs - and thanks i saw you merged some of them! Also to flag: this one github.com/MultiQC/MultiQC/pull/3632 is from a colleague and also concerns mosdepth. And i see some later fix related to mosdepth too: github.com/MultiQC/MultiQC/pull/3660 (not by us). If you prefer to group review work per tool you might want to look at those in batch 🙂
p
What would be helpful is if you can give me a list of the PRs that are important for you to be merged into this release please
🫡 1
Then I can add them to the milestone
🙏 1
I will probably spend a night or two at some point doing a burst of activity and a release, so if they are queued I will make sure to include the,
❤️ 1
~Two~Three are sorted already right? So just these 2 now? 1. github.com/MultiQC/MultiQC/pull/3632 2. github.com/MultiQC/MultiQC/pull/3612
k
HI! and thanks! yes these two and this one: github.com/MultiQC/MultiQC/pull/3615
👍🏻 1
these three PRs - 3632, 3612 and 3615 is the ones we are waiting for. We can assist you with whatever you need, as much as we can!
p
I've triggered Claude reviews on all three claude code
🚀 1
Take with a pinch of salt - don't feel like you have to agree / do everything it proposes. Sometimes what it says is rubbish, or taking guidelines I've set as hard and fast rules.
✍️ 1
But it can give a useful starting point and does often find a bunch of the same stuff that I would find, so lets you tackle that before I put eyes on it.
💯 1
🫡 1