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): Give more idea why we are suggesting an arg #4798

Merged
merged 1 commit into from Mar 28, 2023

Conversation

epage
Copy link
Member

@epage epage commented Mar 28, 2023

One challenge with this is finding something that generally works. Making this work perfectly for one setting will make it inconsistent with other settings and take up more binary size / compile time.

So in the end, I felt like just mirroring rustc (with a bit more brevity) seemed like a decent experiment. This will be evaluated by the feedback on release.

This is a small part of #4638

One challenge with this is finding something that generally works.
Making this work perfectly for one setting will make it inconsistent
with other settings and take up more binary size / compile time.

So in the end, I felt like just mirroring rustc (with a bit more
brevity) seemed like a decent experiment.  This will be evaluated by the
feedback on release.

This is a small part of clap-rs#4638
@epage epage merged commit 53cb165 into clap-rs:master Mar 28, 2023
20 of 21 checks passed
@epage epage deleted the similar branch March 28, 2023 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant