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

Required ArgGroup displayed incorrenctly #476

Closed
panicbit opened this issue Apr 10, 2016 · 1 comment
Closed

Required ArgGroup displayed incorrenctly #476

panicbit opened this issue Apr 10, 2016 · 1 comment
Labels
C-bug Category: Updating dependencies

Comments

@panicbit
Copy link
Contributor

The last character in the usage message of ArgGroups seems to always be missing.

For example this code

App::new("demo")
    .arg(Arg::with_name("foo").long("foo"))
    .arg(Arg::with_name("bar").long("bar"))
    .group(ArgGroup::with_name("options").required(true).args(&[
        "foo", "bar"
    ]))
    .get_matches();

displays the group as [--foo|--ba]

@kbknapp
Copy link
Member

kbknapp commented Apr 10, 2016

As I said in the PR, thanks for finding, filing, and fixing this! 👍

@kbknapp kbknapp mentioned this issue Apr 10, 2016
@kbknapp kbknapp added C-bug Category: Updating dependencies P2: need to have labels Apr 10, 2016
homu added a commit that referenced this issue Apr 10, 2016
homu added a commit that referenced this issue Apr 10, 2016
homu added a commit that referenced this issue Apr 11, 2016
homu added a commit that referenced this issue Apr 11, 2016
@homu homu closed this as completed in #479 Apr 11, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Category: Updating dependencies
Projects
None yet
Development

No branches or pull requests

2 participants