https://nf-co.re logo
Join Slack
Powered by
Is it ok to transfer seqsubmit pipeline to nf-core...
# team-maintainers
e
Is it ok to transfer seqsubmit pipeline to nf-core already? https://github.com/EBI-Metagenomics/seqsubmit A minimal test run of two modules + MultiQC already runs (when ENA is not down). Would like to have its tasks on the hackathon project board
👀 1
🙌🏽 1
m
for me the sooner the better
🙏 1
Can I get admin access to this repo?
e
Let me ask MArtin
m
maxulysse
Copy code
Transfer ownership
Organization members cannot transfer repositories
e
Asked him to do it
image.png
Deadlock!
m
<!subteam^S016E36N7TQ> ok to give martin temporary rights so he can transfer #C09F9HR0A9E to nf-core?
👍🏻 1
👍 2
A closed door is only an opportunity to use an open window
🔏 1
🤣 2
trying to figure out the proper settings from my side
ok I think it should be ok now
🤞
e
ok Martin transfered it to you
m
yeah, that was probably easier
thanks
e
let's gooo
I also set the branch protection rules for main and dev, but couldnt do the TEMPLATE one. Can you do that, and also make sure that main and dev ones are actually working properly?
m
e
and make me and Martin co-admin 😄
✅ 1
m
i will use our standard branch rulesets, replacing yours (which are inactive for some reason)
🙏 2
e
Thanks, even better
m
wonderful
thanks @mashehu I was about to ask you for this
m
done
also fixed the team permissions
🙏 1
do you want to have dev as default branch until you release?
e
sure, let's do it by the book
I still cannot view the branch rules here: https://github.com/nf-core/seqsubmit/settings/branches as I can with proteinfamilies for example: https://github.com/nf-core/proteinfamilies/settings/branches
m
we are slowly moving over to rulesets instead of the classic branch protection
e
Oooh I see, should I change for proteinfamilies likewise then?
m
no urgent need, but sure
(just be aware that for proteinfamilies there should be status checks requirements for 
dev
, because it is released)
e
I would put status checks on seqsubmit as well, because people might accidentely push and merge stuff during hackathon..what do you think?
m
sure, we just are usually a bit more relaxed for pre-release to not make people loose the momentum, but the sooner it is added the better 🙂
do you then also want to have PR reviews enforced for PRs to dev?
e
as an admin, you can always force push 🤫
I'd say 1 review yes
👍🏻 1
m
perfect
2 Views
Open in Slack
PreviousNext
Powered by