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

MissingParameterException not thrown for missing mandatory @Parameter when options are specified #106

Closed
remkop opened this issue Apr 19, 2017 · 0 comments
Milestone

Comments

@remkop
Copy link
Owner

remkop commented Apr 19, 2017

The pre-check in processPositionalParameters0 verifies only that there are at least as many command line arguments as there are mandatory @parameters. It does not yet know how many of these arguments will be consumed as @options.

In the Interpreter.parse(Stack) method, the final check on the requiredFields set passes because this set only contains mandatory @options, not @parameters.

@remkop remkop added this to the 0.9.2 milestone Apr 19, 2017
@remkop remkop closed this as completed in b837081 Apr 19, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant