Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

basic clap cli #186

Closed
wants to merge 2 commits into from
Closed

basic clap cli #186

wants to merge 2 commits into from

Conversation

debris
Copy link
Collaborator

@debris debris commented Jan 19, 2016

based on #185

uses clap cli

  • cli is defined in cli.yml
  • log level is set using --verbosity option instead of RUST_LOG env variable

@debris debris added the A0-pleasereview 🤓 Pull request needs code review. label Jan 19, 2016
@arkpar
Copy link
Collaborator

arkpar commented Jan 19, 2016

conflicted

@arkpar
Copy link
Collaborator

arkpar commented Jan 19, 2016

Have you seen this crate: https://github.com/docopt/docopt.rs? It uses a compiler plugin to generate args struct out of description, which is nicer than having a runtime parser and key-value access interface

@debris
Copy link
Collaborator Author

debris commented Jan 19, 2016

hmm... you may be right.
I will check it out.

@gavofyork
Copy link
Contributor

I'm now working on a docopt PR.

@gavofyork gavofyork added A4-gotissues 💥 Pull request is reviewed and has significant issues which must be addressed. and removed A0-pleasereview 🤓 Pull request needs code review. labels Jan 20, 2016
@debris debris closed this Jan 20, 2016
@debris debris deleted the cli-clap branch January 31, 2016 23:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A4-gotissues 💥 Pull request is reviewed and has significant issues which must be addressed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants