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.

Sign up for GitHub

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

Jump to bottom

fix(parser): Resolve problems around allow_hyphen_values #4187

Merged
merged 12 commits into from Sep 7, 2022
Merged

fix(parser): Resolve problems around allow_hyphen_values #4187

merged 12 commits into from Sep 7, 2022

Commits on Sep 6, 2022

  1. docs: Improve description around trailing_var_arg

    @epage
    epage committed Sep 6, 2022
    Copy the full SHA
    9a4c2bc View commit details
    Browse the repository at this point in the history

Commits on Sep 7, 2022

  1. feat: Allow specifying trailing_var_arg on Arg, like last 

    This has been a bit out of place being on the command.  Now its clearer
what the user intends to be the trailing var arg and it is more likely
to be discovered.
    @epage
    epage committed Sep 7, 2022
    Copy the full SHA
    b07d02e View commit details
    Browse the repository at this point in the history

  2. fix: Deprecate Command::trailing_var_arg 

    Now that we have it on `Arg`, we don't need it on `Command`
    @epage
    epage committed Sep 7, 2022
    Copy the full SHA
    bffce7f View commit details
    Browse the repository at this point in the history

  3. fix(parser): Make Command/Arg behave same for shorts with allow_hyphe… 

    …n_values
    @epage
    epage committed Sep 7, 2022
    Copy the full SHA
    bc5c5e4 View commit details
    Browse the repository at this point in the history

  4. refactor(parser): Highlight short/long parallels

    @epage
    epage committed Sep 7, 2022
    Copy the full SHA
    94a5e7f View commit details
    Browse the repository at this point in the history

  5. fix(parser): Arg::allow_hyphen_values correctly handles long args in … 

    …first pos

This makes it match up with `Command::allow_hyphen_values` which was the
guiding factor for what the behavior should be.

This supersedes clap-rs#4039

Fixes clap-rs#3880
Fixes clap-rs#1538
    @emersonford @epage
    emersonford authored and epage committed Sep 7, 2022
    Copy the full SHA
    fd9a5a1 View commit details
    Browse the repository at this point in the history

  6. fix(derive): Make Command::allow_hyphen_values forward to Arg

    @epage
    epage committed Sep 7, 2022
    Copy the full SHA
    7a2bbca View commit details
    Browse the repository at this point in the history

  7. fix(derive): Deprecate Command::allow_hyphen_values 

    Fixes clap-rs#3450
    @epage
    epage committed Sep 7, 2022
    Copy the full SHA
    d45e4be View commit details
    Browse the repository at this point in the history

  8. docs: Cross-link allow_hyphe_values with trailing_var_arg

    @epage
    epage committed Sep 7, 2022
    Copy the full SHA
    2ebe5ae View commit details
    Browse the repository at this point in the history

  9. perf: Shrink the code

    @epage
    epage committed Sep 7, 2022
    Copy the full SHA
    16d5a2a View commit details
    Browse the repository at this point in the history

  10. feat: Allow specifying negative nums on Arg, like hyphens

    @epage
    epage committed Sep 7, 2022
    Copy the full SHA
    8ad29ef View commit details
    Browse the repository at this point in the history

  11. fix: Deprecate Command::allow_negative_numbers 

    Better to set on individual args
    @epage
    epage committed Sep 7, 2022
    Copy the full SHA
    1258f3e View commit details
    Browse the repository at this point in the history