Hello all, I am looking for two reviewers for big ...
# release-review-trading
k
Hello all, I am looking for two reviewers for big release 😄 Happy to reciprocate now or in future https://github.com/nf-core/variantbenchmarking/pull/253
👀 1
n
I can take a look next Wednesday if no one has done it by then. Please remind me if necessary 😛
k
sure 🎉
n
Also calling @James Fellows Yates to see who's next on the list of core team reviewers 😉
extreme teamwork 1
m
I volunteer too
❤️ 1
🎉 1
f
I just had a brief look and I did not see from the Changelog why this is a 2.0.0 and not a 1.3.0 🙂
m
proud versionning?
🫣 1
k
I thought it is suitable to have version 2 now, as I added a number of new features and I dont have much more to add for a while.. but of course it is debatable :D
f
Ah ok because I thought we only bump the major version when we introduce breaking changes :)
k
I can make the changes breaking old 😛 just joking.. I donno i can also bump version 1.4. Since I want to submit paper draft with this I think I wanted to have a nice version
🤣 1
f
Just pointing at the guidelines here I think this is not a 2.0.0 but a 1.4.0 😅 I'll have a more in depth look after lunch!
m
I can make the changes breaking
🤣
k
I am going back to 1.4.0 😄 I was doubting my proud versioning either :D
🤣 1
j
Just came here to express my joy at folks following the semver guidelines we wrote! nf core party
nf core party 2
k
😄 it is my pleasure 😆 can someone also approve this for going back to 1.4.0 https://github.com/nf-core/variantbenchmarking/pull/255
✏️ 1
j
Btw @Nicolas Vannieuwkerke I am definitely not on the hook for this one 1. This isn't a first release 😉 2. According to our tracking board, Edmun, Fran, Jose, Júlia, Yourself, Phil nor Sateesh have done their reviews in the first round 😉
🤣 1
n
Yeah I was not assigning you 😉 I was asking who was next on the review list 😛
(and I have done several first release reviews in the last months 😮)
j
Oh I should totally learn how ot read 😆
By bad!
n
No worries 😉