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

CLAP CLI Parser #81

Closed
Byron opened this issue Apr 24, 2015 · 1 comment
Closed

CLAP CLI Parser #81

Byron opened this issue Apr 24, 2015 · 1 comment

Comments

@Byron
Copy link
Owner

Byron commented Apr 24, 2015

As docopt doesnt provide nice error messages, nor good help messages, we should use clap-rs to alleviate that.

Byron added a commit that referenced this issue Apr 28, 2015
This allows us to setup clap and see if it compiles, which is the prime
goal of the current workflow step.

Related to #81
Byron added a commit that referenced this issue Apr 29, 2015
We are currently setting everything up at runtime, and manage to get
nearly all information into it, except for the more complex
`-u (simple|resumable) <file> <mime>` flag.

Fixes #87
Related to #81
Byron added a commit that referenced this issue Apr 29, 2015
We can't have the `-u <mode> <file> <mime>` style yet, but
clap-rs/clap#88 might help with that
at some point.

Related to #92 and #81
Byron added a commit that referenced this issue Apr 29, 2015
Discovery API now builds and seems to work even ! More testing
will have to be done though to be sure.

Related #81
Byron added a commit that referenced this issue Apr 29, 2015
Mainly minor cleanup, and handling of generator branches that
didn't show up in smaller APIs that were used during the first steps.

related to #81
Byron added a commit that referenced this issue Apr 30, 2015
* `--version` now includes the API revision we embody
  (using crate_version())
* Allow multiple scopes to be specified, instead of just one. Previously
  this was problemantic due to argument parsing of docopt being greedy.
  However, this also means we have to specify the `-r` flag for each
  invocation. See clap-rs/clap#89 .
* Adapted to new signature of `Arg::possible_values()` and used the
  previously orphaned `UploadProtocol` enum.
* Deduplicated code a little by adding the new `opt_values()` generator
  function.

 Related to #81
@Byron Byron closed this as completed in d0ce221 Apr 30, 2015
Byron added a commit that referenced this issue Apr 30, 2015
* adjust documentation to resemble actual upload flag semantics. It was
  still using the one previously used in docopt.
* Make -m <mime-type> optional, defaulting to 'application/octet-stream'

Should have been fixed alongside of #81
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant