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

Use ArgSettings to store Arg::exclusive #3072

Closed
epage opened this issue Dec 7, 2021 · 3 comments · Fixed by #3105
Closed

Use ArgSettings to store Arg::exclusive #3072

epage opened this issue Dec 7, 2021 · 3 comments · Fixed by #3105
Labels
A-builder Area: Builder API C-bug Category: Updating dependencies E-easy Call for participation: Experience needed to fix: Easy / not much E-help-wanted Call for participation: Help is requested to fix this issue. E-medium Call for participation: Experience needed to fix: Medium / intermediate 💸 $5

Comments

@epage
Copy link
Member

epage commented Dec 7, 2021

Clap: master

Context: to reduce memory overhead, we pack all settings but this one isn't packed.
@epage epage added C-bug Category: Updating dependencies C: settings labels Dec 7, 2021
@pksunkara pksunkara added 💸 $5 D: easy E-help-wanted Call for participation: Help is requested to fix this issue. E-easy Call for participation: Experience needed to fix: Easy / not much E-medium Call for participation: Experience needed to fix: Medium / intermediate labels Dec 8, 2021
@epage epage added A-builder Area: Builder API and removed C: settings labels Dec 8, 2021
@colinwahl
Copy link
Contributor

Hi! I'd like to give this a try.

@colinwahl colinwahl mentioned this issue Dec 9, 2021
Merged
@pksunkara
Copy link
Member

@colinwahl Feel free to redeem the bounty as defined here

@colinwahl
Copy link
Contributor

I can forgo the bounty on this - thanks though

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-builder Area: Builder API C-bug Category: Updating dependencies E-easy Call for participation: Experience needed to fix: Easy / not much E-help-wanted Call for participation: Help is requested to fix this issue. E-medium Call for participation: Experience needed to fix: Medium / intermediate 💸 $5
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use ArgSettings to store Arg::exclusive colinwahl/clap
3 participants
@epage @pksunkara @colinwahl