-
Notifications
You must be signed in to change notification settings - Fork 66
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
Better CLI implementation #51
Comments
Definitely. This would be a huge improvement. |
@JakeStanger does this resolve the issue adequately? |
Absolutely, that's fantastic service! Awesome and very speedy work @drupol |
Thanks! It's the second time I do a PR for a rust project, I really like it ! |
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The current CLI is fine, but the need to be in the current directory for some operations is a little annoying. The current implementation also makes it difficult to expand for more features (such as the
WatchConfig
fields added in #49 ).I would suggest initially migrating the current code to use Clap or similar.
Commands could then be updated to the something like the below format:
The text was updated successfully, but these errors were encountered: