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

Add check/3, parse_and_check/3, and complete test cases #25

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

saleyn
Copy link
Contributor

@saleyn saleyn commented Jul 23, 2013

The functions now allow to specify options:
[help | {skip, [Name::atom()]}]

help - instructs getopt:check/3 to return 'help'
    when "-h" command line option is given and
    option specification contains definition of 'help'.
    This is needed to short-circuit the checker of
            required options when help is requested.

{skip, Names} - tells the checker to skip checking given options

Added test cases for format_error/2 and the new functions mentioned above

The functions now allow to specify options:
    [help | {skip, [Name::atom()]}]

	help - instructs getopt:check/3 to return 'help'
		when "-h" command line option is given and
		option specification contains definition of 'help'.
		This is needed to short-circuit the checker of
                required options when help is requested.

	{skip, Names} - tells the checker to skip checking given options

Added test cases for format_error/2 and the new functions mentioned above
@saleyn
Copy link
Contributor Author

saleyn commented Sep 19, 2013

ping

1 similar comment
@ghost
Copy link

ghost commented Dec 12, 2013

ping

@jcomellas
Copy link
Owner

I'm not sure I like this completely. Let me analyze it for a bit and I will have a response tomorrow.

@saleyn
Copy link
Contributor Author

saleyn commented Dec 20, 2013

Did you get a chance to look at the patch?

@ghost
Copy link

ghost commented Jun 28, 2016

ping?

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