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

feat(derive): Don't require Display for default ArgEnum #3188

Merged
merged 2 commits into from Dec 16, 2021
Merged

feat(derive): Don't require Display for default ArgEnum #3188

merged 2 commits into from Dec 16, 2021

Commits on Dec 16, 2021

  1. fix: Loosen lifetimes on PossibleValue

    @epage
    epage committed Dec 16, 2021
    Configuration menu
    Copy the full SHA
    dee3adb View commit details
    Browse the repository at this point in the history

  2. feat(derive): Don't require Display for default ArgEnum 

    While I'm unsure how much type specialization we should do, we
intentionally have the `arg_enum` attribute for doing special behavior
based on it, so let's take advantage of it.

Fixes clap-rs#3185
    @epage
    epage committed Dec 16, 2021
    Configuration menu
    Copy the full SHA
    8924dd7 View commit details
    Browse the repository at this point in the history