Skip to content

Implement new functionality suggested in Issue 283#286

Merged
ericnewton76 merged 5 commits intocommandlineparser:developfrom
e673:issue283
Aug 13, 2018
Merged

Implement new functionality suggested in Issue 283#286
ericnewton76 merged 5 commits intocommandlineparser:developfrom
e673:issue283

Conversation

@e673
Copy link
Copy Markdown
Contributor

@e673 e673 commented May 5, 2018

When a property throws an exception when settings a value, it is treated as parsing error.

@ericnewton76
Copy link
Copy Markdown
Member

This pull request addresses Issue #283

Just adding this comment so Github actually links them together properly

Copy link
Copy Markdown
Member

@ericnewton76 ericnewton76 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work! Thanks a bunch. I'll let it sit for a bit to gain exposure and then merge it in.

@ericnewton76 ericnewton76 changed the base branch from master to develop May 11, 2018 18:42
@ericnewton76 ericnewton76 modified the milestones: 2.2 patches, 2.3.0 May 11, 2018
@ericnewton76 ericnewton76 merged commit 3b825c3 into commandlineparser:develop Aug 13, 2018
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.

2 participants