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

fix(error): Be more accurate in smart usage #3390

Merged
merged 1 commit into from Feb 2, 2022
Merged

Commits on Feb 2, 2022

  1. fix(error): Be more accurate in smart usage

    For some errors, we use the unroll logic to get the list of required
    arguments.  The usage then does the same, but without a matcher.  This
    was causing the lists to not match.
    
    As a side effect, this fixed an ordering issue where we were putting the
    present arg after the not-present arg.  I assume its because we ended up
    reporting the items twice but the first time is correctly ordered and
    gets precedence.
    
    This was split out of clap-rs#3020
    epage committed Feb 2, 2022
    Copy the full SHA
    06aa418 View commit details
    Browse the repository at this point in the history