-
Notifications
You must be signed in to change notification settings - Fork 2
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
add README and LICENSE #48
Conversation
@BobrImperator let me know if you have any feedback as well – mainly interested in whether anything substantial is missing that should be in a README. There will be guides going into much more depth anyway. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ideas:
pace
without arguments should output--help
by default- It'd be awesome to include the commands and their explanation in generated project's README. Currently it only has a project's name.
- I think that the blueprint is missing a common place for starting other server dependencies. If I needed to create a background task, I'm forced to repeat the setup in both tests and the actual server.
Questions:
- Would it be OK to add the
cli/
crate to.gitignore
?
It causes quite a bit of bloat when searching up files.
Alternatively, we could probably communicate that this is something the users won't have to touch anymore and they're free to exclude it themselves.
Yes, that's to be done still
I think that's something for a later release. There's quite some questions coming with that, e.g. what do you use as a persistent queue for a background tasks implementation.
No since we want that to be checked in with the project. I understand you want to exclude it from your editor but I don't think adding it to |
Co-authored-by: Bartlomiej Dudzik <[email protected]>
see #20