Skip to content
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: BcubeVTK v0.1.0 #120345

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JuliaRegistrator
Copy link
Contributor

@JuliaRegistrator JuliaRegistrator commented Nov 28, 2024

Copy link
Contributor

github-actions bot commented Nov 28, 2024

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 registration

Please make sure that you have read the package naming guidelines.

2. AutoMerge Guidelines are all met! ✅

Your new package registration met all of the guidelines for auto-merging and is scheduled to be merged when the mandatory waiting period (3 days) has elapsed.

3. To pause or stop registration

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 [noblock] in your comment.

Tip: You can edit blocking comments to add [noblock] in order to unblock auto-merging.

UUID: f7a61acb-f482-4505-b2eb-e9cadd76b6b4
Repo: https://github.com/bcube-project/BcubeVTK.jl.git
Tree: 7a22431b75549ebfea2f39fd862c3cca46358cc2

Registrator tree SHA: 17aec322677d9b81cdd6b9b9236b09a3f1374c6a
@goerz
Copy link
Member

goerz commented Nov 29, 2024

Thank you for submitting your package! Could you add a little bit of documentation before registering? At the very least, in addition to the description that's already there, that would be a small usage example in the README. An important part of packages in General is that any potential user can figure out what the package is about and how to get started with using it. That is really difficult when there is no documentation.

In the longer term, I definitely recommend setting up a Documenter-based documentation. Before a v1.0 release, or for smaller packages that can be effectively described entirely with their README, that's not a requirement, though.

@JuliaTagBot JuliaTagBot added the AutoMerge: last run blocked by comment PR blocked by one or more comments lacking the string [noblock]. label Nov 29, 2024
@bmxam
Copy link

bmxam commented Nov 29, 2024

Hi Michael,

I agree. I will update the README with more info and a short example. However, I won't push anything before monday (but anyway I guess the PR is blocked until I push something).

Just for your information, this is actually a "plugin" for the main package Bcube.jl who is instrumented with Documenter & Cie.

@goerz
Copy link
Member

goerz commented Nov 29, 2024

I won't push anything before monday (but anyway I guess the PR is blocked until I push something.

Yes, that's right. No rush!

Just for your information, this is actually a "plugin" for the main package Bcube.jl who is instrumented with Documenter & Cie.

Sometimes it's good enough to have a brief README with just a paragraph that points people to the documentation of the main project. The only point is that looking at the README, someone should have some idea of the context and if/how it can be used.

[noblock]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AutoMerge: last run blocked by comment PR blocked by one or more comments lacking the string [noblock]. new package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants