-
Notifications
You must be signed in to change notification settings - Fork 472
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
New package: BarBay v0.0.1 #93121
New package: BarBay v0.0.1 #93121
Conversation
JuliaRegistrator
commented
Oct 9, 2023
- Registering package: BarBay
- Repository: https://github.com/mrazomej/BarBay.jl
- Created by: @mrazomej
- Version: v0.0.1
- Commit: b707c33fbeb074415c4fe0839a18abd79e01b134
- Reviewed by: @mrazomej
- Reference: mrazomej/BarBay.jl@b707c33#commitcomment-129553932
- Description: Repository for the BarBay Julia package for Bayesian inference of relative fitness on barcode sequencing data.
UUID: 3e4a4fdd-3c42-44bf-ab95-e818c96fb7a6 Repo: https://github.com/mrazomej/BarBay.jl.git Tree: ba674afd22304fc2eb6dd133ccc93106c998b913 Registrator tree SHA: f73a20c99934db92a256057d0d83ba394036a701
Your Since you are registering a new package, please make sure that you have read the package naming guidelines: https://julialang.github.io/Pkg.jl/dev/creating-packages/#Package-naming-guidelines-1 If you want to prevent this pull request from being auto-merged, simply leave a comment. If you want to post a comment without blocking auto-merging, you must include the text |
Yes. I must apologize for the inconvenience. My collaborators made this name change request after I had registered BayesFitness. If possible, I would like to delete BayesFitness and retain only the BarBay.jl package. |
I am not sure if that is possible. Let's wait for some other opinions on that. |
Since it's only been three days, it would probably be okay to either merge this PR and make another PR deleting [noblock] |
As the uuid changed between the packages, I think reverting #92941 would be the cleanest. But unfortunately I cannot find the button to start a PR reverse PR |
I've created #93308 to revert the registration of BayesFitness, but please don't take that necessarily as approval of this plan. We should discuss a bit more whether it's ok or not to delete it. |
I sincerely apologize for this inconvenience. I literally got the name change request from my collaborators the day I submitted the registration for the previous package name. I never imagined it would cause such difficulties. |
Wouldn't that have given you the three-day waiting period to block the merge? But in any case, I see no reason not to merge #93308 (but I don't have merge permissions myself) Just next time, make sure you're on the same page with your collaborators before registering a package. ;-) [noblock] |
This situation shows how much of a novice I am in all this stuff. I googled how to change the name, and I understood it already had to be registered to do so. I didn't even think about trying to revert the other submission. Lesson learned. And thanks for your help and patience. |
[noblock] I changed all my comments above to be non-blocking. |
Dear Julia team, Could you let me know if my situation has been updated? I submitted a publication that has this Thank you so much in advance for your help, and I apologize again for the complication. |
[noblock] Looks like #93308 has been merged, so this PR here can now proceed, right? Every comment will have to be edited to contain |
[noblock] Or just ask @StefanKarpinski to merge manually, since he has a blocking comment anyway |
@StefanKarpinski, I'm sorry to bother you with this situation again. @goerz mentioned I should ask you to merge the package manually. This package accompanies a publication I recently submitted for peer review, so it would be great if the reviewers could access the package as easily as possible. I don't know if this was done. But today, I tried installing
and it says the package is not registered. |
@lgoettgens @goerz @StefanKarpinski, sorry to be insistent, but as I mentioned earlier, this package is part of a publication I submitted for peer review. Therefore, please help me register the package. Thanks in advance for your help. |
I asked on Slack for someone with the appropriate permissions to merge this 🤞 |