-
Notifications
You must be signed in to change notification settings - Fork 461
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: RankMS v0.1.0 #116327
base: master
Are you sure you want to change the base?
New package: RankMS v0.1.0 #116327
Conversation
JuliaRegistrator
commented
Sep 30, 2024
- Registering package: RankMS
- Repository: https://github.com/yanjer/RankMS.jl
- Created by: @yanjer
- Version: v0.1.0
- Commit: a8038995cf5423e3cb0394a0a41d8ac82046bec9
- Git reference: HEAD
UUID: 64d803ca-a168-423f-ba94-4afacd1091d3 Repo: https://github.com/yanjer/RankMS.jl.git Tree: 4fc01668d95277eb01521610240b8324e002d82e Registrator tree SHA: 191228b6dd8b9d0e2965ae3e705fe54c51dcfee8
Hello, I am an automated registration bot. I help manage the registration process by checking your registration against a set of AutoMerge guidelines. If all these guidelines are met, this pull request will be merged automatically, completing your registration. It is strongly recommended to follow the guidelines, since otherwise the pull request needs to be manually reviewed and merged by a human. 1. New package registrationPlease make sure that you have read the package naming guidelines. 2. AutoMerge Guidelines which are not met ❌
3. Needs action: here's what to do next
If you need help fixing the AutoMerge issues, or want your pull request to be manually merged instead, please post a comment explaining what you need help with or why you would like this pull request to be manually merged. Then, send a message to the 4. To pause or stop registrationIf 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 Tip: You can edit blocking comments to add |
There is no problem, why still can't merge, please help merge.[noblock] |
Please stop spamming the registry with repeated registration requests which all fail for the same reason, and read the results of the automerge action:
I.e. your package is not working. You should make sure that it actually works (you can check for yourself by adding it from GitHub and then Apart from that as @goerz said in one of your earlier attempts, long and descriptive package names are favoured in the Julia General registry. |
Please make sure to test your package locally before registration. One thing that stands out as a potential issue is that you have a Your After you commit the fixes, retrigger the registration by commenting To be honest, this package needs a major clean-up before you should try re-registering. While you're at it, I would also recommend fixing the Markdown formatting in your README. You should be using "```" for multiline code blocks. |
@nilshg One small detail in @yanjer's favor: the "AutoMerge" step was "skipped" (presumably due to some more severe underlying issue like the presence of a Downloading the package and just trying to instantiate it reveals a lot of issues, though. So @yanjer should be able to sort this problems out by testing the package locally. |
I believe it's more of a GitHub presentation issue. If you go to the Btw, having a |
Good to know! I came across this because I tried to instantiate the package after cloning it, and that gave me errors about dependencies not being in the Manifest. So at the very least, the Manifest is outdated, which seems like a bad idea ;-) |
I was not aware of this! It would probably be good to put those instructions into the bot message, where it says "See the AutoMerge logs for details." I always assumed that was referring to the list of checks at the bottom of page, but the Checks tab is much more informative! |
The list at the bottom has been useful in the past, but when I look at the current set of PRs it seems somewhat random how well it correlates with the |