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

Ignore default value for required parameter #567

Merged
merged 2 commits into from
Aug 29, 2023
Merged

Ignore default value for required parameter #567

merged 2 commits into from
Aug 29, 2023

Conversation

gwenn
Copy link
Contributor

@gwenn gwenn commented Aug 28, 2023

Fix #566

@mkarg mkarg merged commit 60df0cf into cbeust:master Aug 29, 2023
1 check passed
@mkarg mkarg self-assigned this Aug 29, 2023
@mkarg mkarg added the bug Bug label Aug 29, 2023
@mkarg
Copy link
Collaborator

mkarg commented Aug 29, 2023

@gwenn Thank you for reporting and fixing this issue! :-)

@gwenn
Copy link
Contributor Author

gwenn commented Aug 29, 2023

@gwenn Thank you for reporting and fixing this issue! :-)

Thank you for helping me !

@gwenn gwenn deleted the ignore_default_value_of_required_parameter branch August 29, 2023 18:02
stuartfehr pushed a commit to stuartfehr/jcommander that referenced this pull request Feb 28, 2024
stuartfehr pushed a commit to stuartfehr/jcommander that referenced this pull request Feb 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Default value should not be validated for required parameter
2 participants