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

feat(derive): Allow type-less fields #4371

Merged
merged 1 commit into from Oct 11, 2022
Merged

feat(derive): Allow type-less fields #4371

merged 1 commit into from Oct 11, 2022

Conversation

epage
Copy link
Member

@epage epage commented Oct 11, 2022

When overriding other fields, help or version flag, globals, etc, a user might not care about the value, so let's ignore the lookup.

Been talking about this for a while but Issue #4367 moved this forward because there wasn't a good way to handle this without changing behavior.

When overriding other fields, help or version flag, globals, etc, a user
might not care about the value, so let's ignore the lookup.

Been talking about this for a while but Issue clap-rs#4367 moved this forward
because there wasn't a good way to handle this without changing
behavior.
@epage epage merged commit a6eddb0 into clap-rs:master Oct 11, 2022
@epage epage deleted the unit branch October 11, 2022 15:42
wfxr added a commit to hackathon-2022-ticli/ticli that referenced this pull request Oct 21, 2022
clap-rs/clap#4367
clap-rs/clap#4371

Signed-off-by: Wenxuan Zhang <wenxuangm@gmail.com>
tmccombs added a commit to tmccombs/fd that referenced this pull request Nov 11, 2022
tmccombs added a commit to tmccombs/fd that referenced this pull request Nov 11, 2022
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