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

Require rules ignore a defaulted value #3076

Closed
2 tasks done
epage opened this issue Dec 7, 2021 · 0 comments · Fixed by #3420
Closed
2 tasks done

Require rules ignore a defaulted value #3076

epage opened this issue Dec 7, 2021 · 0 comments · Fixed by #3420
Assignees
@pksunkara
Labels
A-validators Area: ArgMatches validation logi C-bug Category: Updating dependencies E-medium Call for participation: Experience needed to fix: Medium / intermediate
Milestone
3.1

Comments

@epage
Copy link
Member

epage commented Dec 7, 2021

  • I have searched the discussions
  • I have searched the existing issues

Description

This consolidates:

An attempt at a fix was reverted in 9753f68. See also #3020
@epage epage added C-bug Category: Updating dependencies C: args labels Dec 7, 2021
epage added a commit to epage/clap that referenced this issue Dec 7, 2021
@epage
Revert "Revert "Revert "docs: Clarify corner caseses with default val…
7b61993 
…ues"""

This reverts commit 98f696c.

This was previously missed.  See clap-rs#3076
epage added a commit to epage/clap that referenced this issue Dec 7, 2021
@epage
Revert "Revert "Revert "docs: Clarify corner caseses with default val…
5ba7562 
…ues"""

This reverts commit 98f696c.

This was previously missed.  See clap-rs#3076
@epage epage added this to the 3.1 milestone Dec 7, 2021
epage added a commit to epage/clap that referenced this issue Dec 7, 2021
@epage
Revert "Revert "Revert "docs: Clarify corner caseses with default val…
d36a981 
…ues"""

This reverts commit 98f696c.

This was previously missed.  See clap-rs#3076
@pksunkara pksunkara self-assigned this Dec 8, 2021
epage added a commit to epage/clap that referenced this issue Dec 8, 2021
@epage
Revert "Revert "Revert "docs: Clarify corner caseses with default val…
a12c8fd 
…ues"""

This reverts commit 98f696c.

This was previously missed.  See clap-rs#3076
epage added a commit to epage/clap that referenced this issue Dec 8, 2021
@epage
Revert "Revert "Revert "docs: Clarify corner caseses with default val…
26bbe60 
…ues"""

This reverts commit 98f696c.

This was previously missed.  See clap-rs#3076
@epage epage added A-builder Area: Builder API A-validators Area: ArgMatches validation logi and removed C: args labels Dec 8, 2021
@epage epage mentioned this issue Dec 8, 2021
Closed
@pksunkara pksunkara linked a pull request Dec 9, 2021 that will close this issue
Requires and Conflicts should ignore default_value #3020
Closed
This was referenced Dec 9, 2021
Closed
Closed
Closed
@epage epage added E-medium Call for participation: Experience needed to fix: Medium / intermediate and removed A-builder Area: Builder API labels Dec 13, 2021
@epage epage modified the milestones: 3.x, 3.1 Feb 2, 2022
@epage epage mentioned this issue Feb 8, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pksunkara pksunkara

Labels
A-validators Area: ArgMatches validation logi C-bug Category: Updating dependencies E-medium Call for participation: Experience needed to fix: Medium / intermediate
Projects
None yet
Milestone
3.1
2 participants
@epage @pksunkara