Do I need a formal approval from nf-core to merge a PR, or if someone from nf-core reviewed it, I fixed all the issues, and the tests pass, I can just queue it for merging?
I'm trying to balance bugging nf-core members and not wasting people's time.
Thank you
m
mashehu
04/29/2026, 7:16 PM
as soon as you have a ✅ and no outstanding comments, you are good to go.
u
Uri David Akavia
04/29/2026, 7:17 PM
Sorry, very literal minded person here. Do you mean when they've pressed approved on Github (and the checkmark appeared), or when they say "looks good to me" (even if they haven't pressed approve)?
m
mashehu
04/29/2026, 7:18 PM
sorry for not being clearer before: yes, approved via the github review approval.
u
Uri David Akavia
04/29/2026, 7:19 PM
Thank you!
s
Simon Pearce
04/29/2026, 7:32 PM
I've reviewed again and given you the ✅ .
Uri David, please just keep hassling if you need things reviewed 😉