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

Regression in error messages when two args from a non-multiple group were passed #1859

Closed
CreepySkeleton opened this issue Apr 24, 2020 · 4 comments · Fixed by #2309
Closed
Labels
E-hard Call for participation: Experience needed to fix: Hard / a lot
Milestone

Comments

@CreepySkeleton
Copy link
Contributor

CreepySkeleton commented Apr 24, 2020

This is what it used to look like:

error: The argument '--delete' cannot be used with '<base>'
USAGE:
    clap-test <base|--delete>
For more information try --help"

After #1856 has landed, it looks like

error: Found argument '--all' which wasn't expected, or isn't valid in this context
If you tried to supply `--all` as a PATTERN use `-- --all`
USAGE:
    clap-test <-a|--delete>
For more information try --help"

This is undesirable and must be corrected, but the bugfix is more important.

@CreepySkeleton CreepySkeleton added C-bug Category: Updating dependencies C: errors E-hard Call for participation: Experience needed to fix: Hard / a lot and removed C-bug Category: Updating dependencies labels Apr 24, 2020
@CreepySkeleton CreepySkeleton added this to the 3.1 milestone Apr 24, 2020
@CreepySkeleton CreepySkeleton changed the title Regression in error messages when two args from non-mulpiple groups were passed Regression in error messages when two args from a non-multiple group were passed Apr 24, 2020
@issuehunt-oss
Copy link

issuehunt-oss bot commented May 28, 2020

@pksunkara has funded $20.00 to this issue.


@issuehunt-oss issuehunt-oss bot added the 💵 Funded on Issuehunt This issue has been funded on Issuehunt label May 28, 2020
@pksunkara
Copy link
Member

@ldm0 Do you want this one next?

@ldm0
Copy link
Member

ldm0 commented Nov 7, 2020

Yep.

This was referenced Jan 20, 2021
@issuehunt-oss
Copy link

issuehunt-oss bot commented Jan 27, 2021

@pksunkara has cancelled funding for this issue.(Cancelled amount: $20.00) See it on IssueHunt

@issuehunt-oss issuehunt-oss bot removed the 💵 Funded on Issuehunt This issue has been funded on Issuehunt label Jan 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
E-hard Call for participation: Experience needed to fix: Hard / a lot
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants