-
Notifications
You must be signed in to change notification settings - Fork 460
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: Carbon v1.0.0 #93438
New package: Carbon v1.0.0 #93438
Conversation
JuliaRegistrator
commented
Oct 15, 2023
•
edited
Loading
edited
- Registering package: Carbon
- Repository: https://github.com/vrkansagara/Carbon.jl
- Created by: @vrkansagara
- Version: v1.0.0
- Commit: 11203da3791766f702943b6c7d9f36c904710a1b
- Reviewed by: @vrkansagara
- Reference: new deployment at JuliaHub vrkansagara/Tz.jl#4 (comment)
- Description: A simple Julia API extension for DateTime.
[noblock] This looks a bit like a test package ... 🤔 |
Your
Note that the guidelines are only required for the pull request to be merged automatically. However, it is strongly recommended to follow them, since otherwise the pull request needs to be manually reviewed and merged by a human. After you have fixed the AutoMerge issues, simple retrigger Registrator, which will automatically update this pull request. You do not need to change the version number in your If you do not want to fix the AutoMerge issues, please post a comment explaining why you would like this pull request to be manually merged. Then, send a message to the Since you are registering a new package, please make sure that you have also 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 |
b00a9f0
to
d6670bf
Compare
@cormullion Please check latest state |
d6670bf
to
0600bd2
Compare
I don't think code like this needs to be in a package? function now()
return DateTime(2023)
end |
0600bd2
to
094e452
Compare
@cormullion updated the code. Package name similar to 2 existing packages. |
094e452
to
49fd746
Compare
There's nothing you can do now - you have to wait 3 days before a submitted package can be registered - if it's considered to be a useful addition to the registry. You can discuss the naming of your package on the Slack channel mentioned above |
What does this package do? How is it related to carbon? |
UUID: c6408494-260b-4f8d-aa04-3305a96ccce5 Repo: https://github.com/vrkansagara/Carbon.jl.git Tree: 1cc09593b66b2b0826175653a03ee721fdea4f66 Registrator tree SHA: f73a20c99934db92a256057d0d83ba394036a701
49fd746
to
d053061
Compare
Here is the explanation of the package name. Why did I choose this package name as There is a very good library in a different language community called Carbon , From where I was inspired to take the name and it has a very friendly implementation of the There is no actual |
I think there are a few problems here:
|
noted with thanks @briochemc , I have just changed the package name. |
@JuliaRegistrator |
FYI, I have changed that package name to |
No, sorry, but it does not work: The new name, "Tz", is just as confusing if not worse. But this does not matter at this stage anyway, because your package is basically empty and thus not ready for registration. |
Now I am really confused about what is the definition of an empty package here.? |
I'm sorry that this is confusing to you. There's no formal definition as far as I know, but your package provides close to zero functionality. It merely contains four "Hello World!"-printing types of functions or aliases:
Anyway, What is the intended purpose of the package? Even this is not clear... |
@JuliaRegistrator Give up this PR and close it |
This pull request has been inactive for 30 days and will be automatically closed 7 days from now. If this pull request should not be closed, please either (1) fix the AutoMerge issues and re-trigger Registrator, which will automatically update the pull request, or (2) post a comment explaining why you would like this pull request to be manually merged. [noblock] |
This pull request has been inactive for more than 30 days and has automatically been closed. Feel free to register your package or version again once you fix the AutoMerge issues. [noblock] |