Skip to content

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

Open
saleyn wants to merge 5 commits into
jcomellas:masterfrom
saleyn:format_error
Open

Add check/3, parse_and_check/3, and complete test cases#25
saleyn wants to merge 5 commits into
jcomellas:masterfrom
saleyn:format_error

Conversation

@saleyn

@saleyn saleyn commented Jul 23, 2013

Copy link
Copy Markdown
Contributor

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

saleyn commented Sep 19, 2013

Copy link
Copy Markdown
Contributor Author

ping

1 similar comment
@ghost

ghost commented Dec 12, 2013

Copy link
Copy Markdown

ping

@jcomellas

Copy link
Copy Markdown
Owner

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

@saleyn

saleyn commented Dec 20, 2013

Copy link
Copy Markdown
Contributor Author

Did you get a chance to look at the patch?

@ghost

ghost commented Jun 28, 2016

Copy link
Copy Markdown

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